Backport(バックポート) #1214
管理画面のナビ設定画面でエラーが表示されない
開始日:
2010-05-26
期日:
進捗率:
100%
説明
関連するチケット
関係しているリビジョン
履歴
#1 tu nguyen ngoc が13年以上前に更新
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を tu nguyen ngoc にセット
#2 tu nguyen ngoc が13年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 50788ef17796b0978099d716d968d3aae9c3d6a6 で適用されました。
#3 Mutsumi Imamura が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
確認しました。2点気になる点があります。
ご検討をお願いします。
- Entry nameに長い文字列が登録できてしまうのは、メンバー側のナビゲーションの表示崩れにつながるので適切な文字数制限を設ける必要があると思いますがいかがでしょうか。
- URLに@hogeや、hoge/huga、http://example.com/ 以外の文字列を登録できてしまうのは問題ないでしょうか。現状、半角数字のみ、全角数字のみ、半角記号のみ、全角文字のみが登録できることを確認しました。
#4 Shinichi Urabe が13年以上前に更新
Imamura Mutsumi は書きました:
確認しました。2点気になる点があります。
ご検討をお願いします。
- Entry nameに長い文字列が登録できてしまうのは、メンバー側のナビゲーションの表示崩れにつながるので適切な文字数制限を設ける必要があると思いますがいかがでしょうか。
これについては、既存のサイトで意図的に長い名前にし、そういったレイアウトにしているSNSが存在する可能性がありますので、現状のままでいきましょう。
- URLに@hogeや、hoge/huga、http://example.com/ 以外の文字列を登録できてしまうのは問題ないでしょうか。現状、半角数字のみ、全角数字のみ、半角記号のみ、全角文字のみが登録できることを確認しました。
こちらについてもユーザー画面に影響はないようなので、このままの仕様でいきましょう
#5 Shinichi Urabe が13年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
テストOKとのことです。今村さん確認
#6 Rimpei Ogawa が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
for ($i=0;$i<$count-1;$i++)
の部分がコーディング規約違反です。
#7 Rimpei Ogawa が13年以上前に更新
$this->forward('navigation','list');
カンマの後にスペースがありません。
#8 Masato Nagasawa が13年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 a5ed67dba3774d3a4595d1a39d10db1c3d215177 で適用されました。
#9 Masato Nagasawa が13年以上前に更新
更新履歴 77aa3c0587a0cce2a54a513b0c27bd9631e99db4 で適用されました。
#10 Rimpei Ogawa が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
- リクエストパラメータの nav[type] に存在しない値を入れるとエラーになります。$params['type'] のチェックを加えてください
if ($request->isMethod('post'))
は edit に合わせてif ($request->isMethod(sfWebRequest::POST))
にしてください
#11 Shinichi Urabe が13年以上前に更新
- 担当者 を tu nguyen ngoc から Masato Nagasawa に変更
#12 Masato Nagasawa が13年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 744618a89162d8214037e4c2ace6e39007790e48 で適用されました。
#13 Mutsumi Imamura が13年以上前に更新
再テストOKです。
#14 Masato Nagasawa が13年以上前に更新
更新履歴 99f185d550624ed209cf475c0c65a92388cc17de で適用されました。
#15 Masato Nagasawa が13年以上前に更新
更新履歴 9bfe10ce6f69ced9451122198207ede188478e78 で適用されました。
#16 Rimpei Ogawa が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
#17 Mutsumi Imamura が13年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
再々テストOKです。