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