ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 13 years ago

Closed 13 years ago

#3655 closed defect (fixed)

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

Reported by: kiwa Owned by: nobody
Priority: minor Milestone: OpenPNE2.13.6
Component: 指定しない Version: 2.14.x
Keywords: Cc:

Description

■現象

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

■原因

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

■修正内容

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

■関連情報

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

Change History (6)

comment:1 Changed 13 years ago by kiwa

Keywords: 確認中 added
  • r10811 … 修正リビジョン
  • r10822 … trunk取り込みリビジョン

comment:2 Changed 13 years ago by shingo

Keywords: 差し戻し added; 確認中 removed
  • 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');
    

comment:3 in reply to:  2 Changed 13 years ago by 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に追記しておきました。

comment:4 in reply to:  2 Changed 13 years ago by kiwa

Keywords: 確認中 added; 差し戻し removed

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で該当する記述が削除されていました。

comment:5 Changed 13 years ago by ShogoKawahara

Keywords: テスト待ち added; 確認中 removed

comment:6 Changed 13 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.