Bug(バグ) #2515
携帯のコミュニティイベントコメントで必須である説明を追記する
開始日:
2011-10-18
期日:
進捗率:
100%
3.6 で発生するか:
Unknown (未調査)
[QA]バグ通知済:
いいえ
3.8 で発生するか:
Unknown (未調査)
opCommunityTopicPlugin 絞込条件:
説明
Overview (現象)¶
携帯のコミュニティイベントページ(/communityEvent/xx)のコメント欄について
コメントの後に * が表示されているが、説明文がないので分かりにくい。
また、コメントのラベル表示方法がコミュニティトピックなどと異なるため違和感がある。
トピック
イベント
Way to fix (修正内容)¶
PC版/コミュニティトピック(携帯版)と同様、「* は必須項目です。」などの文言追記
および、ラベル表示をコミュニティトピックと合わせる
(参考)PC版での修正: http://trac.openpne.jp/changeset/13535#file0
関連するチケット
履歴
#1 Maki Takahashi が12年以上前に更新
- ファイル 111018-0002.png 表示 を追加
- ファイル 111018-0001.png 表示 を追加
- 説明 を更新 (diff)
#2 Maki Takahashi が12年以上前に更新
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を Maki Takahashi にセット
#3 Maki Takahashi が12年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
#4 Yuya Watanabe が約12年前に更新
- 対象バージョン を 1.0.x にセット
- 3.6 で発生するか を Unknown (未調査) にセット
- 3.4 で発生するか を Unknown (未調査) にセット
#5 Kousuke Ebihara が約12年前に更新
- 対象バージョン を 1.0.x から 1.0.4 に変更
#6 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 {
#7 Maki Takahashi がほぼ12年前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
#8 Yuya Watanabe がほぼ12年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
レビューOKとして下記コミット群で適用しました.
https://github.com/tejimaya/opCommunityTopicPlugin/compare/673175fe75...393746a267
#9 Yuma Sakata がほぼ12年前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
テストOKです。