Bug(バグ) #2515
携帯のコミュニティイベントコメントで必須である説明を追記する
Start date:
2011-10-18
Due date:
% Done:
100%
3.6 で発生するか:
Unknown (未調査)
[QA]バグ通知済:
No
3.8 で発生するか:
Unknown (未調査)
opCommunityTopicPlugin 絞込条件:
Description
Overview (現象)¶
携帯のコミュニティイベントページ(/communityEvent/xx)のコメント欄について
コメントの後に * が表示されているが、説明文がないので分かりにくい。
また、コメントのラベル表示方法がコミュニティトピックなどと異なるため違和感がある。
トピック
イベント
Way to fix (修正内容)¶
PC版/コミュニティトピック(携帯版)と同様、「* は必須項目です。」などの文言追記
および、ラベル表示をコミュニティトピックと合わせる
(参考)PC版での修正: http://trac.openpne.jp/changeset/13535#file0
Related issues
History
#1
Updated by Maki Takahashi over 11 years ago
- File 111018-0002.png View added
- File 111018-0001.png View added
- Description updated (diff)
#2
Updated by Maki Takahashi about 11 years ago
- Status changed from New(新規) to Accepted(着手)
- Assignee set to Maki Takahashi
#3
Updated by Maki Takahashi about 11 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
#4
Updated by Yuya Watanabe almost 11 years ago
- Target version set to 1.0.x
- 3.6 で発生するか set to Unknown (未調査)
- 3.4 で発生するか set to Unknown (未調査)
#5
Updated by Kousuke Ebihara almost 11 years ago
- Target version changed from 1.0.x to 1.0.4
#6
Updated by Yuya Watanabe almost 11 years ago
- Status changed from Pending Review(レビュー待ち) to 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
Updated by Maki Takahashi almost 11 years ago
- Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
#8
Updated by Yuya Watanabe almost 11 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
レビューOKとして下記コミット群で適用しました.
https://github.com/tejimaya/opCommunityTopicPlugin/compare/673175fe75...393746a267
#9
Updated by Yuma Sakata almost 11 years ago
- Status changed from Pending Testing(テスト待ち) to Fixed(完了)
- % Done changed from 70 to 100
テストOKです。