Backport（バックポート） #1253
管理画面でコミュニティカテゴリを削除した場合、コミュニティトップのカテゴリ表示が空欄になる
100%
説明
現象¶
管理画面でコミュニティカテゴリを削除した場合、コミュニティトップのカテゴリ表示が空欄になる。
この状態でコミュニティ編集画面に遷移すると、カテゴリ欄にカテゴリidが一番小さい値のものになってしまい、無意識のうちにコミュニティカテゴリを変更してしまう可能性がある。
再現環境¶
- OpenPNE3.0.8-dev
- OpenPNE3.2.5-dev
- OpenPNE3.4.4-dev
- OpenPNE3.5.3-dev
仕様案¶
カテゴリが削除された時、その削除したカテゴリを使用しているコミュニティのカテゴリを未分類とする。
これにより、カテゴリが空欄の状態でコミュニティ編集をした際に無意識のうちに他のカテゴリに変更してしまうというリスクは回避できるかと思いますがどうでしょうか？
原因¶
カテゴリが削除された場合の分岐処理がテンプレート側にはいっていない
修正内容¶
- 管理画面で大カテゴリを削除した気に小カテゴリがレコードに残っているので、getNode() から delete() を呼ぶように修正
- コミュニティ作成編集時にコミュニティカテゴリがない場合、表示しないように修正
- コミュニティの表示時に、カテゴリがない場合、表示しないように修正（PC,携帯）
関連するチケット
関係しているリビジョン
履歴
#1
Mutsumi Imamura がほぼ14年前に更新
- 対象バージョン を OpenPNE 3.4.6 から OpenPNE 3.4.7 に変更
#2
tu nguyen ngoc が13年以上前に更新
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を tu nguyen ngoc にセット
#3
tu nguyen ngoc が13年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
更新履歴 b9ec94e9a1a6db8bdc51866133fca50ff7affc68 で適用されました。
#4
Shinichi Urabe が13年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
- 担当者 を tu nguyen ngoc から Shinichi Urabe に変更
取り消して次バージョンにて対応
#5
Shinichi Urabe が13年以上前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
更新履歴 0d9d34f89d7e71d6742f81fe4e2e4def36c0e63f で適用されました。
#6
Shinichi Urabe が13年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Fixing（修正待ち） に変更
- 担当者 を Shinichi Urabe から tu nguyen ngoc に変更
- 対象バージョン を OpenPNE 3.4.7 から OpenPNE 3.4.8 に変更
#7
Shinichi Urabe が13年以上前に更新
- 対象バージョン を OpenPNE 3.4.8 から OpenPNE 3.4.9 に変更
#8
Shinichi Urabe が13年以上前に更新
- 対象バージョン を OpenPNE 3.4.9 から OpenPNE 3.4.10 に変更
#9
tu nguyen ngoc が約13年前に更新
- ステータス を Pending Fixing（修正待ち） から Pending Review（レビュー待ち） に変更
更新履歴 976193307a935418bcc376137cea57f951835dc0 で適用されました。
#10
Shinichi Urabe が約13年前に更新
9e7fec86de1ec075b089c7cd79c722fa940990a0 のモバイルの修正も含んでいます （チケット番号間違いました）
#11
Masato Nagasawa が約13年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
原因と修正内容の記述をお願いします。
#12
Shinichi Urabe が約13年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
#13
Masato Nagasawa が約13年前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
管理画面で大カテゴリを削除した気に小カテゴリがレコードに残っているので、getNode() から delete() を呼ぶように修正
この修正以前に大カテゴリを削除した場合、小カテゴリが残ったままであるということでしょうか？
そうであれば、存在しない大カテゴリidの小カテゴリを削除するマイグレーションファイルを作成する必要があると思います。
#14
Masato Nagasawa が約13年前に更新
管理画面で大カテゴリを削除した気に小カテゴリがレコードに残っているので、getNode() から delete() を呼ぶように修正
この修正は #1951 で対応する方向になりました。
これに関連する修正の取り消しをお願いします。
#15
Yuma Sakata が約13年前に更新
テスト結果に問題なかったのですが、「項目追加」ボタンのレイアウトが気になりました。
<input value=" 項目追加 " type="submit">
こちらのような感じにして上の「確定」ボタンと「削除」ボタンのラインに合わせる方が綺麗だと思うのですが、どうでしょうか？
#16
Shinichi Urabe が約13年前に更新
坂田 雄麻 は書きました:
テスト結果に問題なかったのですが、「項目追加」ボタンのレイアウトが気になりました。
[...]
こちらのような感じにして上の「確定」ボタンと「削除」ボタンのラインに合わせる方が綺麗だと思うのですが、どうでしょうか？
こちらはこのバグの問題と違うので、機能追加として新規に作成していただければと思います
Masato Nagasawa は書きました:
管理画面で大カテゴリを削除した気に小カテゴリがレコードに残っているので、getNode() から delete() を呼ぶように修正
この修正は #1951 で対応する方向になりました。
これに関連する修正の取り消しをお願いします。
これについて、さっき、チャットでの内容と異なるのですが（すみません）、大カテゴリを削除した場合、小カテゴリが残り、このチケットの問題の解決とならないので、この対処が入るまでの問題を #1951（親チケット #1952）で対処することで検討したいと思っています
#17
Shinichi Urabe が約13年前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
#18
Yuma Sakata が約13年前に更新
「項目追加」ボタンのレイアウトに関しては別チケット #1953 で対応します。
#20
Masato Nagasawa が約13年前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
不要なレコードが残る問題の対処は #1951 で行うということで了解しました。
#21
Mutsumi Imamura が約13年前に更新
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
テスト済みですので完了とします。