Project

General

Profile

Bug（バグ） #2515

携帯のコミュニティイベントコメントで必須である説明を追記する

Added by Maki Takahashi over 9 years ago. Updated about 9 years ago.

Status:
Fixed（完了）
Priority:
Normal（通常）
Assignee:
Maki Takahashi
Target version:
1.0.4
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

111018-0002.png View (12 KB) Maki Takahashi, 2011-10-18 20:48

111018-0001.png View (11.1 KB) Maki Takahashi, 2011-10-18 20:48

Related issues

Related to opCommunityTopicPlugin - Bug（バグ） #643: 参加登録にコメント記入が必須である旨が、記載されていない Fixed（完了） 2010-02-15
Related to opCommunityTopicPlugin - Bug（バグ） #2857: 携帯版でイベント参加登録にコメント記入が必須である旨が記載されていない Invalid（無効） 2012-03-06

History

#1 Updated by Maki Takahashi over 9 years ago

#2 Updated by Maki Takahashi over 9 years ago

  • Status changed from New（新規） to Accepted（着手）
  • Assignee set to Maki Takahashi

#3 Updated by Maki Takahashi over 9 years ago

  • Status changed from Accepted（着手） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

#4 Updated by Yuya Watanabe about 9 years ago

  • Target version set to 1.0.x
  • 3.6 で発生するか set to Unknown (未調査)
  • 3.4 で発生するか set to Unknown (未調査)

#5 Updated by Kousuke Ebihara about 9 years ago

  • Target version changed from 1.0.x to 1.0.4

#6 Updated by Yuya Watanabe about 9 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 about 9 years ago

  • Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）

https://github.com/martini2002jp/opCommunityTopicPlugin/commit/ddc713b710cfb8fbd21693851e89e1ba776665f3
にて、フィードバックに従い空白を削除いたしました。

#8 Updated by Yuya Watanabe about 9 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 about 9 years ago

  • Status changed from Pending Testing（テスト待ち） to Fixed（完了）
  • % Done changed from 70 to 100

テストOKです。

Also available in: Atom PDF