プロジェクト

全般

プロフィール

Bug(バグ) #2515

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

Maki Takahashi12年以上前に追加. ほぼ12年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
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

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

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


関連するチケット

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

履歴

#1 Maki Takahashi12年以上前に更新

#2 Maki Takahashi12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Maki Takahashi にセット

#3 Maki Takahashi12年以上前に更新

  • ステータス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(レビュー待ち) に変更

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

#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です。

他の形式にエクスポート: Atom PDF