コミュニティカテゴリ設定で小カテゴリが正しく登録できない

[kiwa]

■現象

コミュニティカテゴリ設定(admin_page_edit_category)にて、小カテゴリ作成時に指定した並び順が反映されない、作成可否が常に「不可」として登録される。

■原因

登録フィールド名が間違っていた（両フィールド共 [ sort_order ] になっていた。

■修正内容

フィールド名を修正し、作成可否欄をプルダウンに変更し、作成時に並び順・作成可否を反映させるようにした。

■関連情報

#3645: 特定のカテゴリをコミュニティ作成時に選べないようにする機能#3063 のバグ修正

[kiwa]

• r10811 … 修正リビジョン
• r10822 … trunk取り込みリビジョン

[shingo]

• 2.13.6のinsert_data.sql・create_tables.sqlがない
• update用SQLファイルのファイル名が2.13.6ではない
• webapp/lib/db/commu.php
  • 古い変数名を活かす形で修正していますが、活かす必要があるのでしょうか?
  • 修正ついでにSQL文の「=」と「.」の縦位置を揃えていただけるとありがたいです。

    1905 
    1906 // 作成可能なコミュニティカテゴリリストを取得
    1907 function db_commu_c_commu_category4createflag()
    1908 {
    1909     return db_commu_c_commu_category4is_create_commu();
    1910 }
    1911 function db_commu_c_commu_category4is_create_commu()
    1912 {
    1913     $is_create_commu = 1;
    1914     $sql = 'SELECT cat.* FROM c_commu_category AS cat' .
    1915             ' INNER JOIN c_commu_category_parent AS pcat' .
    1916             ' USING (c_commu_category_parent_id)' .
    1917             ' WHERE cat.is_create_commu = ?' .
    1918             ' ORDER BY pcat.sort_order, cat.sort_order';
    1919     $params = array(intval($is_create_commu));
    1920     return db_get_all($sql,$params);
    1921 }
    

    1940 /**
    1941  * コミュニティを作成できるカテゴリかどうか
    1942  *
    1943  * @param int $c_commu_category_id
    1944  * @return bool
    1945  */
    1946 function db_commu_c_commu_category_is_create_flag($c_commu_category_id)
    1947 {
    1948     return db_commu_c_commu_category_is_create_commu($c_commu_category_id);
    1949 }
    1950 function db_commu_c_commu_category_is_create_commu($c_commu_category_id)
    1951 {
    1952     $sql = 'SELECT is_create_commu FROM c_commu_category'
    1953          . ' WHERE c_commu_category_id = ?';
    1954     $params = array(intval($c_commu_category_id));
    1955     return db_get_one($sql, $params);
    1956 }
    

• intvalは不要ではないでしょうか

  1919     $params = array(intval($is_create_commu));
  

• 「,」のあとに空白を入れてください。

  webapp/modules/pc/page/c_edit.php
  
  33	            openpne_redirect('pc','page_h_err_c_edit');
  

[kiwa]

Replying to shingo:

• 2.13.6のinsert_data.sql・create_tables.sqlがない
• update用SQLファイルのファイル名が2.13.6ではない

update sqlのリネームはr10823で行っています。報告漏れ失礼しました。

また、2.13.6のセットアップsqlの作成は#3660での対応になります。r10811のセットアップsqlの変更を取り込むようdescriptionに追記しておきました。

[kiwa]

Replying to shingo:

sqlファイル以外の指摘について修正しました。

1. r10875: db_commu_c_commu_category4createflagの使用をやめ、db_commu_c_commu_category4is_create_commuに統一
2. r10876: 作成可能コミュニティカテゴリリスト取得時の不要なintvalを削除
3. r10877: この機能での追加分に入っていたSQL文の「=」と「.」の縦位置をそろえた

3については他にも多くの箇所で統一されてないので、完全に統一するとしたら別チケットで作業するべきかと思います。

「,」のあとに空白を入れてください。

こちらはr10841で該当する記述が削除されていました。