Project

General

Profile

Backport(バックポート) #1253

管理画面でコミュニティカテゴリを削除した場合、コミュニティトップのカテゴリ表示が空欄になる

Added by Mutsumi Imamura about 9 years ago. Updated over 8 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-05-12
Due date:
% Done:

100%


Description

現象

管理画面でコミュニティカテゴリを削除した場合、コミュニティトップのカテゴリ表示が空欄になる。
この状態でコミュニティ編集画面に遷移すると、カテゴリ欄にカテゴリidが一番小さい値のものになってしまい、無意識のうちにコミュニティカテゴリを変更してしまう可能性がある。

再現環境

  • OpenPNE3.0.8-dev
  • OpenPNE3.2.5-dev
  • OpenPNE3.4.4-dev
  • OpenPNE3.5.3-dev

仕様案

カテゴリが削除された時、その削除したカテゴリを使用しているコミュニティのカテゴリを未分類とする。
これにより、カテゴリが空欄の状態でコミュニティ編集をした際に無意識のうちに他のカテゴリに変更してしまうというリスクは回避できるかと思いますがどうでしょうか?

原因

カテゴリが削除された場合の分岐処理がテンプレート側にはいっていない

修正内容

  • 管理画面で大カテゴリを削除した気に小カテゴリがレコードに残っているので、getNode() から delete() を呼ぶように修正
  • コミュニティ作成編集時にコミュニティカテゴリがない場合、表示しないように修正
  • コミュニティの表示時に、カテゴリがない場合、表示しないように修正(PC,携帯)

Related issues

Related to OpenPNE 3 - Bug(バグ) #1055: 管理画面でコミュニティカテゴリを削除した場合、コミュニティトップのカテゴリ表示が空欄になる Fixed(完了) 2010-05-12
Related to OpenPNE 3 - Bug(バグ) #1951: #1055 で管理画面で大カテゴリを削除した場合に小カテゴリがレコードに残らないように対処しているが、その対処が入る前に残った小カテゴリのデータの対処がされていない Invalid(無効) 2011-03-15
Related to OpenPNE 3 - Enhancement(機能追加・改善) #1953: 管理画面のプロフィール項目設定の「項目追加」ボタンのレイアウトについて New(新規) 2011-03-15

Associated revisions

Revision b9ec94e9 (diff)
Added by tu nguyen ngoc about 9 years ago

(fixes #1253, BP from #1055) do not display category entry in case catefory is empty

Revision 0d9d34f8 (diff)
Added by Shinichi Urabe almost 9 years ago

Revert "(fixes #1253, BP from #1055) do not display category entry in case catefory is empty"

This reverts commit b9ec94e9a1a6db8bdc51866133fca50ff7affc68.

Revision 97619330 (diff)
Added by tu nguyen ngoc over 8 years ago

(fixes #1253, BP from #1055) do not display category entry in case catefory is empty

History

#1 Updated by Mutsumi Imamura about 9 years ago

  • Target version changed from OpenPNE 3.4.6 to OpenPNE 3.4.7

#2 Updated by tu nguyen ngoc about 9 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to tu nguyen ngoc

#3 Updated by tu nguyen ngoc about 9 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 b9ec94e9a1a6db8bdc51866133fca50ff7affc68 で適用されました。

#4 Updated by Shinichi Urabe almost 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Assignee changed from tu nguyen ngoc to Shinichi Urabe

取り消して次バージョンにて対応

#5 Updated by Shinichi Urabe almost 9 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 0d9d34f89d7e71d6742f81fe4e2e4def36c0e63f で適用されました。

#6 Updated by Shinichi Urabe almost 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Fixing(修正待ち)
  • Assignee changed from Shinichi Urabe to tu nguyen ngoc
  • Target version changed from OpenPNE 3.4.7 to OpenPNE 3.4.8

#7 Updated by Shinichi Urabe almost 9 years ago

  • Target version changed from OpenPNE 3.4.8 to OpenPNE 3.4.9

#8 Updated by Shinichi Urabe almost 9 years ago

  • Target version changed from OpenPNE 3.4.9 to OpenPNE 3.4.10

#9 Updated by tu nguyen ngoc over 8 years ago

  • Status changed from Pending Fixing(修正待ち) to Pending Review(レビュー待ち)

更新履歴 976193307a935418bcc376137cea57f951835dc0 で適用されました。

#10 Updated by Shinichi Urabe over 8 years ago

9e7fec86de1ec075b089c7cd79c722fa940990a0 のモバイルの修正も含んでいます (チケット番号間違いました)

#11 Updated by Masato Nagasawa over 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

原因と修正内容の記述をお願いします。

#12 Updated by Shinichi Urabe over 8 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

#13 Updated by Masato Nagasawa over 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

管理画面で大カテゴリを削除した気に小カテゴリがレコードに残っているので、getNode() から delete() を呼ぶように修正

この修正以前に大カテゴリを削除した場合、小カテゴリが残ったままであるということでしょうか?
そうであれば、存在しない大カテゴリidの小カテゴリを削除するマイグレーションファイルを作成する必要があると思います。

#14 Updated by Masato Nagasawa over 8 years ago

管理画面で大カテゴリを削除した気に小カテゴリがレコードに残っているので、getNode() から delete() を呼ぶように修正

この修正は #1951 で対応する方向になりました。
これに関連する修正の取り消しをお願いします。

#15 Updated by Yuma Sakata over 8 years ago

テスト結果に問題なかったのですが、「項目追加」ボタンのレイアウトが気になりました。

<input value="  項目追加  " type="submit">

こちらのような感じにして上の「確定」ボタンと「削除」ボタンのラインに合わせる方が綺麗だと思うのですが、どうでしょうか?

#16 Updated by Shinichi Urabe over 8 years ago

坂田 雄麻 は書きました:

テスト結果に問題なかったのですが、「項目追加」ボタンのレイアウトが気になりました。

[...]

こちらのような感じにして上の「確定」ボタンと「削除」ボタンのラインに合わせる方が綺麗だと思うのですが、どうでしょうか?

こちらはこのバグの問題と違うので、機能追加として新規に作成していただければと思います

Masato Nagasawa は書きました:

管理画面で大カテゴリを削除した気に小カテゴリがレコードに残っているので、getNode() から delete() を呼ぶように修正

この修正は #1951 で対応する方向になりました。
これに関連する修正の取り消しをお願いします。

これについて、さっき、チャットでの内容と異なるのですが(すみません)、大カテゴリを削除した場合、小カテゴリが残り、このチケットの問題の解決とならないので、この対処が入るまでの問題を #1951(親チケット #1952)で対処することで検討したいと思っています

#17 Updated by Shinichi Urabe over 8 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

#18 Updated by Yuma Sakata over 8 years ago

「項目追加」ボタンのレイアウトに関しては別チケット #1953 で対応します。

#19 Updated by Shinichi Urabe over 8 years ago

Masato Nagasawa は書きました:

管理画面で大カテゴリを削除した気に小カテゴリがレコードに残っているので、getNode() から delete() を呼ぶように修正

この修正は #1951 で対応する方向になりました。
これに関連する修正の取り消しをお願いします。

同じ問題の #1394 のバックポートではFIXになっており、このチケットだけ #1055#1394 と違う修正を
加えると、追加した #1951 でバージョンにより対処方法が異なると複雑になるため、現状のままにしておきたいと考えています。

#20 Updated by Masato Nagasawa over 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

不要なレコードが残る問題の対処は #1951 で行うということで了解しました。

#21 Updated by Mutsumi Imamura over 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テスト済みですので完了とします。

Also available in: Atom PDF