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