Project

General

Profile

Backport(バックポート) #1214

管理画面のナビ設定画面でエラーが表示されない

Added by Mutsumi Imamura about 11 years ago. Updated about 11 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2010-05-26
Due date:
% Done:

100%


Description

現象

管理画面のナビ設定で、URL等を空欄のままsubmitを押しても必須項目ですエラーが表示されない。
追加時、編集時どちらともにエラーが表示されない。

再現環境

  • OpenPNE3.5.x
  • OpenPNE3.4.x
  • OpenPNE3.2.x

原因

修正内容


Related issues

Related to OpenPNE 3 - Bug(バグ) #1108: 管理画面のナビ設定画面でエラーが表示されない Fixed(完了) 2010-05-26
Related to OpenPNE 3 - Backport(バックポート) #1540: 管理画面のナビ設定画面でエラーが表示されない Fixed(完了) 2010-05-26

Associated revisions

Revision 50788ef1 (diff)
Added by tu nguyen ngoc about 11 years ago

(fixes #1214, BP from #1108) show error message when input is empty

Revision a5ed67db (diff)
Added by Masato Nagasawa about 11 years ago

fixed the coding standard (fixes #1214, BP from #1108)

Revision 77aa3c05 (diff)
Added by Masato Nagasawa about 11 years ago

fixed the coding standard (fixes #1214, BP from #1108)

Revision 744618a8 (diff)
Added by Masato Nagasawa about 11 years ago

fixed navi_type check, 'post' to sfWebRequest::POST (fixes #1214, BP from #1108)

Revision 99f185d5 (diff)
Added by Masato Nagasawa about 11 years ago

fixed notice of the array access (fixes #1214, BP from #1108)

Revision 9bfe10ce (diff)
Added by Masato Nagasawa about 11 years ago

fixed notice of the array access (fixes #1214, BP from #1108)

History

#1 Updated by tu nguyen ngoc about 11 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to tu nguyen ngoc

#2 Updated by tu nguyen ngoc about 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 50788ef17796b0978099d716d968d3aae9c3d6a6 で適用されました。

#3 Updated by Mutsumi Imamura about 11 years ago

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

確認しました。2点気になる点があります。
ご検討をお願いします。

  • Entry nameに長い文字列が登録できてしまうのは、メンバー側のナビゲーションの表示崩れにつながるので適切な文字数制限を設ける必要があると思いますがいかがでしょうか。
  • URLに@hogeや、hoge/huga、http://example.com/ 以外の文字列を登録できてしまうのは問題ないでしょうか。現状、半角数字のみ、全角数字のみ、半角記号のみ、全角文字のみが登録できることを確認しました。

#4 Updated by Shinichi Urabe about 11 years ago

Imamura Mutsumi は書きました:

確認しました。2点気になる点があります。
ご検討をお願いします。

  • Entry nameに長い文字列が登録できてしまうのは、メンバー側のナビゲーションの表示崩れにつながるので適切な文字数制限を設ける必要があると思いますがいかがでしょうか。

これについては、既存のサイトで意図的に長い名前にし、そういったレイアウトにしているSNSが存在する可能性がありますので、現状のままでいきましょう。

  • URLに@hogeや、hoge/huga、http://example.com/ 以外の文字列を登録できてしまうのは問題ないでしょうか。現状、半角数字のみ、全角数字のみ、半角記号のみ、全角文字のみが登録できることを確認しました。

こちらについてもユーザー画面に影響はないようなので、このままの仕様でいきましょう

#5 Updated by Shinichi Urabe about 11 years ago

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

テストOKとのことです。今村さん確認

#6 Updated by Rimpei Ogawa about 11 years ago

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

for ($i=0;$i<$count-1;$i++) の部分がコーディング規約違反です。

#7 Updated by Rimpei Ogawa about 11 years ago

$this->forward('navigation','list'); カンマの後にスペースがありません。

#8 Updated by Masato Nagasawa about 11 years ago

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

更新履歴 a5ed67dba3774d3a4595d1a39d10db1c3d215177 で適用されました。

#9 Updated by Masato Nagasawa about 11 years ago

更新履歴 77aa3c0587a0cce2a54a513b0c27bd9631e99db4 で適用されました。

#10 Updated by Rimpei Ogawa about 11 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 about 11 years ago

  • Assignee changed from tu nguyen ngoc to Masato Nagasawa

#12 Updated by Masato Nagasawa about 11 years ago

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

更新履歴 744618a89162d8214037e4c2ace6e39007790e48 で適用されました。

#13 Updated by Mutsumi Imamura about 11 years ago

再テストOKです。

#14 Updated by Masato Nagasawa about 11 years ago

更新履歴 99f185d550624ed209cf475c0c65a92388cc17de で適用されました。

#15 Updated by Masato Nagasawa about 11 years ago

更新履歴 9bfe10ce6f69ced9451122198207ede188478e78 で適用されました。

#16 Updated by Rimpei Ogawa about 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#17 Updated by Mutsumi Imamura about 11 years ago

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

再々テストOKです。

Also available in: Atom PDF