Project

General

Profile

Backport(バックポート) #3660

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

Added by isao sano almost 7 years ago. Updated over 6 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2014-07-15
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 d2146f70 (diff)
Added by 誠二 天重 almost 7 years ago

fixed sortable of the community categories (refs #3660, BP from #1859)

History

#1 Updated by isao sano almost 7 years ago

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

#2 Updated by 誠二 天重 almost 7 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • Assignee set to 誠二 天重
  • % Done changed from 0 to 50

#3 Updated by Shinichi Urabe almost 7 years ago

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

レビューOKです

#4 Updated by isao sano over 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

単体テスト完了です。ソース反映待ちです。

以下のバグが見つかりましたが、別チケット対応と致します。

コミュニティカテゴリの大分類がcommunity_categoryのsort_orderでソートされない
https://redmine.openpne.jp/issues/3678

コミュニティカテゴリの小分類の並びが管理画面ではsort_orderではなくIDを参照している
https://redmine.openpne.jp/issues/3679

#5 Updated by Shinichi Urabe over 6 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF