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

Opened 12 years ago

Closed 12 years ago

#3063 closed enhancement (fixed)

特定のカテゴリをコミュニティ作成時に選べないようにする機能

Reported by: nakasone Owned by: nakasone
Priority: minor Milestone: OpenPNE2.13.4
Component: 指定しない Version:
Keywords: Cc:

Description (last modified by nakasone)

■概要

特定のカテゴリをコミュニティ作成時に選べないようにする機能。

■仕様

■関連情報

http://sns.openpne.jp/?m=pc&a=page_c_topic_detail&target_c_commu_topic_id=4070

r8519にて、toranekoさんから提供していただいたコードの取り込み。

Change History (21)

comment:1 Changed 12 years ago by nakasone

Owner: changed from nobody to nakasone
Status: newassigned

取り込みを行います。

comment:2 Changed 12 years ago by nakasone

Keywords: 確認中 added
  • r8640: r8519をマージ

上記の作業をtrunkで行いました。

comment:3 Changed 12 years ago by nakasone

Description: modified (diff)

comment:4 Changed 12 years ago by kiwa

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

browser:OpenPNE/branches/work/toraneko/prj_newfeature/doc/prj_commu_adm.txt

「コミュ管理機能改善プロジェクト改修内容」を確認したところ、DB変更が必要なようです。update sqlの追加をおねがいします。

comment:5 Changed 12 years ago by ebihara

Owner: changed from nakasone to ebihara
Status: assignednew

update用のSQL追加については海老原が受け持ちます。

comment:6 Changed 12 years ago by ebihara

Owner: changed from ebihara to nakasone

以下のリビジョンでupdate用のSQLを追加しました。

comment:7 Changed 12 years ago by ebihara

「新機能を作る会」コミュニティで提案した内容を共有します。

28: えびはらこーすけ 削除
#3063 (特定のカテゴリをコミュニティ作成時に選べないようにする機能)
ttp://trac.openpne.jp/ticket/3063

について提案させていただきます。

提案1. 作成可能なコミュニティカテゴリがひとつもない場合については、「現在コミュニティを作成することができません」といった表示があったほうが、利用者が混乱しなくて済むかなと思いました。
提案2. bool 値が入るとわかっているフィールド名、変数名については、 is_*** という形式にするとそのフィールド、変数の役割が自明になりますのでオススメです。

また、バグとおぼしき現象を見つけたので報告します。
コミュニティカテゴリによるコミュニティ作成の「可」・「不可」設定が、表示上のみしか有効でなく、アクション内では一切考慮していないように見えますがどうでしょうか。

チケットの方にも共有しておきます。

29: えびはらこーすけ 削除
あ、あと webapp/lib/db/deprecated_lib.php についてですが、ファイル名からも自明なように、このファイルに定義されている関数は非推奨であり新規に利用することを想定していません。
2.12開発時に webapp/lib/db/deprecated_lib.php 内で定義されている関数をコールしていたものを、すべて正規の関数のコールに変更したため、 OpenPNE 本体からはこの関数のコールをおこなっている箇所はありません。

しかしながら、以前の OpenPNE を参考にして作ったモジュールや拡張については、この非推奨の関数を使っている可能性を否定しきれません。そのため、 webapp/lib/db/deprecated_lib.php 内に、新関数のラッパーとしてまとめて定義しています。
ということで、互換性の関係上、このファイルに定義されている関数については、削除等しないように留意していただければと思います。

comment:8 Changed 12 years ago by ebihara

修正するべき点は多々ありますが、とりあえず以下の問題についての対処をおこなった段階でリリースをおこないます。

以下以外にも致命的な問題が発覚し、かつ、早急の修正が難しい場合、このチケットで取り込まれた機能については trunk から一時的に外し、リリースします。

  • webapp/lib/db/deprecated_lib.php 内の関数の削除を取り消し
  • コミュニティカテゴリによるコミュニティ作成の「可」・「不可」設定が、表示上のみしか有効でない

comment:9 Changed 12 years ago by shingo

Owner: changed from nakasone to shingo
Status: newassigned

作業します。

comment:10 Changed 12 years ago by shingo

  • webapp/lib/db/deprecated_lib.php 内の関数の削除を取り消し
  • コミュニティカテゴリによるコミュニティ作成の「可」・「不可」設定が、表示上のみしか有効でない

上記2点について、以下のリビジョンで修正しました。

comment:11 Changed 12 years ago by shingo

Keywords: 確認中 added; 差し戻し removed
Owner: changed from shingo to nakasone
Status: assignednew

comment:12 Changed 12 years ago by ebihara

  • 携帯でカテゴリ選択なしでコミュニティが作成できてしまう現象
  • 編集時に無条件でカテゴリが選択できてしまう現象

についても修正しましょうかね。

comment:13 Changed 12 years ago by ebihara

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

comment:14 Changed 12 years ago by shingo

Owner: changed from nakasone to shingo

comment:15 Changed 12 years ago by shingo

以下のリビジョンで修正しました。ご確認ください。

新しいバグを見つけましたのでチケットを作りました。

  • #3155:コミュニティ編集画面にて重複したコミュニティ名を登録できてしまう。

comment:16 Changed 12 years ago by shingo

【訂正】

#3155:コミュニティ編集画面にて重複したコミュニティ名を登録できてしまう。

正しくは #3135 です。

comment:17 Changed 12 years ago by shingo

Keywords: 確認中 added; 差し戻し removed
Owner: changed from shingo to nakasone

comment:18 Changed 12 years ago by ebihara

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

携帯版では WORD_COMMUNITY を使わないでください。

comment:19 Changed 12 years ago by shingo

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

以下のリビジョンで修正しました。ご確認ください。

comment:20 Changed 12 years ago by ebihara

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

comment:21 Changed 12 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: newclosed

このチケットはこれで閉じます。バグは別チケットを作成して潰していきましょう。

Note: See TracTickets for help on using tickets.