操作
Bug（バグ） #2515
完了
携帯のコミュニティイベントコメントで必須である説明を追記する
開始日:
2011-10-18
期日:
進捗率:
100%
予定工数:
3.6 で発生するか:
Unknown (未調査)
［QA］バグ通知済:
いいえ
3.8 で発生するか:
opCommunityTopicPlugin 絞込条件:
説明
Overview (現象)¶
携帯のコミュニティイベントページ(/communityEvent/xx)のコメント欄について
コメントの後に * が表示されているが、説明文がないので分かりにくい。
また、コメントのラベル表示方法がコミュニティトピックなどと異なるため違和感がある。
トピック
イベント
Way to fix (修正内容)¶
PC版/コミュニティトピック（携帯版）と同様、「* は必須項目です。」などの文言追記
および、ラベル表示をコミュニティトピックと合わせる
（参考）PC版での修正： http://trac.openpne.jp/changeset/13535#file0
ファイル
Maki Takahashi さんが約13年前に更新
- ファイル 111018-0002.png 111018-0002.png を追加
- ファイル 111018-0001.png 111018-0001.png を追加
- 説明 を更新 (差分)
Maki Takahashi さんが約13年前に更新
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を Maki Takahashi にセット
Maki Takahashi さんが約13年前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
Yuya Watanabe さんがほぼ13年前に更新
- 対象バージョン を 1.0.x にセット
- 3.6 で発生するか を Unknown (未調査) にセット
- 3.4 で発生するか を Unknown (未調査) にセット
Yuya Watanabe さんが12年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
"%0% is required field" という文言はコア側に含まれていることを確認しました．
現在のアプリケーションを取得する方法が sfConfig::get('sf_app') と sfContext::getInstance()->getApplication() とで2つあるのでどちらを使うべきか迷いましたが，本体側では sfConfg::get('sf_app') が多く使われているため問題ないとしておきます．
差し戻し理由¶
行末に空白がある
lib/form/doctrine/PluginCommunityEventCommentForm.class.php 36行目
35 }
36 else
37 {
Maki Takahashi さんが12年以上前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
Yuya Watanabe さんが12年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
レビューOKとして下記コミット群で適用しました．
https://github.com/tejimaya/opCommunityTopicPlugin/compare/673175fe75...393746a267
Yuma Sakata さんが12年以上前に更新
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
テストOKです。
操作