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