Project

General

Profile

Actions
Copy link

Bug（バグ） #2515

closed

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

Added by Maki Takahashi over 13 years ago. Updated almost 13 years ago.

Status:
Fixed（完了）
Priority:
Normal（通常）
Assignee:
Maki Takahashi
Target version:
1.0.4
Start date:
2011-10-18
Due date:
% Done:

100%

Estimated time:
3.6 で発生するか:
Unknown (未調査)
［QA］バグ通知済:
No
3.8 で発生するか:
opCommunityTopicPlugin 絞込条件:

Description

Overview (現象)

携帯のコミュニティイベントページ(/communityEvent/xx)のコメント欄について
コメントの後に * が表示されているが、説明文がないので分かりにくい。
また、コメントのラベル表示方法がコミュニティトピックなどと異なるため違和感がある。

トピック

イベント

Way to fix (修正内容)

PC版/コミュニティトピック（携帯版）と同様、「* は必須項目です。」などの文言追記
および、ラベル表示をコミュニティトピックと合わせる
（参考）PC版での修正： http://trac.openpne.jp/changeset/13535#file0

Files

Download all files
111018-0002.png (12 KB) 111018-0002.png Maki Takahashi, 2011-10-18 20:48
111018-0001.png (11.1 KB) 111018-0001.png Maki Takahashi, 2011-10-18 20:48

Related issues 2 (0 open — 2 closed)

Related to opCommunityTopicPlugin - Bug（バグ） #643: 参加登録にコメント記入が必須である旨が、記載されていないFixed（完了）洋一郎 櫻井2010-02-15

Actions
Related to opCommunityTopicPlugin - Bug（バグ） #2857: 携帯版でイベント参加登録にコメント記入が必須である旨が記載されていないInvalid（無効）Yuya Watanabe2012-03-06

Actions
Actions
Copy link
 #2

Updated by Maki Takahashi about 13 years ago

  • Status changed from New（新規） to Accepted（着手）
  • Assignee set to Maki Takahashi
Actions
Copy link
 #3

Updated by Maki Takahashi about 13 years ago

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

Updated by Yuya Watanabe almost 13 years ago

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

Updated by Kousuke Ebihara almost 13 years ago

  • Target version changed from 1.0.x to 1.0.4
Actions
Copy link
 #6

Updated by Yuya Watanabe almost 13 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     {

Actions
Copy link
 #7

Updated by Maki Takahashi almost 13 years ago

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

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

Actions
Copy link
 #8

Updated by Yuya Watanabe almost 13 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

Actions
Copy link
 #9

Updated by Yuma Sakata almost 13 years ago

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

テストOKです。

Actions
Copy link

Also available in: Atom PDF