Project

General

Profile

Backport(バックポート) #3635

The community category is not considered a sort of parent categories (コミュニティカテゴリが大カテゴリのソートを考慮していない)

Added by 誠二 天重 about 5 years ago. Updated about 5 years ago.

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

100%


Description

Overview (現象)

管理画面でカテゴリ関係を下記のように定義したとする.

  • A
    • a
    • b
  • B
    • c
    • d
    • e

これを下記の順番にしたとする.

  1. B
    1. c
    2. d
    3. e
  2. A
    1. a
    2. b

このときに,フロントエンド側のコミュニティ新規作成画面などでコミュニティカテゴリを選択する際に表示される select タグの option の項目が c d e a b と表示されるべきだが,a b c d e と表示されてしまう.

Causes (原因)

大カテゴリを考慮して小カテゴリのソートを行っていない。

Way to fix (修正内容)

小カテゴリと大カテゴリを別々のクエリで取得後、PHP 側でソート処理を行う。
(2系ではJOINを使用して1クエリで取得していたが、Doctrineを使用した場合テーブルの構造上の問題で同じ方法は使えない)
この結果を Doctrine_Collection で返すメソッドを CommunityCategoryTable に追加し、
既存のカテゴリ表示を行っている箇所をすべて置換する。


Related issues

Copied from OpenPNE 3 - Bug(バグ) #1859: The community category is not considered a sort of parent categories (コミュニティカテゴリが大カテゴリのソートを考慮していない) Won't fix(対応せず) 2010-12-21

Associated revisions

Revision f4b70b11 (diff)
Added by isao sano about 5 years ago

fixed sortable of the community categories (fixes #3635, BP from #1859)

Revision 9f22af88
Added by Shinichi Urabe about 5 years ago

Merge pull request #149 from isaosano/T-3635

fixed sortable of the community categories (fixes #3635, BP from #1859)

History

#1 Updated by 誠二 天重 about 5 years ago

  • Copied from Bug(バグ) #1859: The community category is not considered a sort of parent categories (コミュニティカテゴリが大カテゴリのソートを考慮していない) added

#2 Updated by isao sano about 5 years ago

下記URLでプルリクしました
https://github.com/openpne/OpenPNE3/pull/149

#3 Updated by isao sano about 5 years ago

  • Status changed from New(新規) to Accepted(着手)

#4 Updated by isao sano about 5 years ago

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

#5 Updated by isao sano about 5 years ago

  • Assignee set to Shinichi Urabe

#6 Updated by isao sano about 5 years ago

  • Assignee changed from Shinichi Urabe to isao sano

#7 Updated by Shinichi Urabe about 5 years ago

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

レビューOKです

#8 Updated by isao sano about 5 years ago

単体試験完了しました。
結合試験終了後ステータスを終了いたします。

#9 Updated by isao sano about 5 years ago

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

結合試験完了しました。
問題ありませんのでステータスを「Fixed」いたします。

#10 Updated by Shinichi Urabe about 5 years ago

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

pull request 取り込み漏れです。。
取り込みました
https://github.com/openpne/OpenPNE3/pull/149

#11 Updated by isao sano about 5 years ago

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

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

#12 Updated by Shinichi Urabe about 5 years ago

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

#13 Updated by Shinichi Urabe about 5 years ago

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

ステータスが戻ったのでなおします

#14 Updated by 誠二 天重 about 5 years ago

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

結合試験完了しました。

Also available in: Atom PDF