Project

General

Profile

Backport(バックポート) #1213

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

Added by Mutsumi Imamura almost 11 years ago. Updated over 10 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-05-26
Due date:
% Done:

100%


Description

現象

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

再現環境

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

原因

エラーの場合の処理がはいっていない

修正内容

エラーの場合、list アクションにforwardし、listアクション内で、bindする処理を追加 テンプレートのフォームでもエラーをrenderするようにも修正


Related issues

Related to OpenPNE 3 - Bug(バグ) #1108: 管理画面のナビ設定画面でエラーが表示されない Fixed(完了) 2010-05-26
Related to OpenPNE 3 - Backport(バックポート) #1956: 管理画面のナビゲーションフォームでナビゲーションのURLとして想定しない文字列が入力された場合、エラーメッセージを表示するように対処する Invalid(無効) 2011-03-17

Associated revisions

Revision 359f6832 (diff)
Added by tu nguyen ngoc almost 11 years ago

(fixes #1213, BP from #1088) i18n of navigation page

Revision 7220f43f (diff)
Added by tu nguyen ngoc almost 11 years ago

Revert "(fixes #1213, BP from #1088) i18n of navigation page"

This reverts commit 359f68328a02bbeb75a5a82a46794936ede23f23.

Revision a8b6e128 (diff)
Added by tu nguyen ngoc almost 11 years ago

(fixes #1213, BP from #1108) fixed: error is not displayed

Revision 41989dac (diff)
Added by Shinichi Urabe almost 11 years ago

Revert "(fixes #1213, BP from #1108) fixed: error is not displayed"

This reverts commit a8b6e128e5142830fbc554a6a8fb8d1c5e4ab05e.

Revision 52cc8127 (diff)
Added by tu nguyen ngoc over 10 years ago

(fixes #1213, BP from #1108) fixed: error is not displayed

History

#1 Updated by Shinichi Urabe almost 11 years ago

  • Priority changed from High(高め) to Normal(通常)

#2 Updated by Mutsumi Imamura almost 11 years ago

  • Target version changed from OpenPNE 3.4.6 to OpenPNE 3.4.7

#3 Updated by tu nguyen ngoc almost 11 years ago

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

#4 Updated by tu nguyen ngoc almost 11 years ago

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

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

#5 Updated by tu nguyen ngoc almost 11 years ago

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

#6 Updated by tu nguyen ngoc almost 11 years ago

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

#7 Updated by Shinichi Urabe almost 11 years ago

  • Assignee changed from tu nguyen ngoc to Shinichi Urabe

#8 Updated by Shinichi Urabe almost 11 years ago

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

取り消して次ぎバージョンで対応

#9 Updated by Shinichi Urabe almost 11 years ago

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

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

#10 Updated by Shinichi Urabe almost 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Fixing(修正待ち)
  • Assignee changed from Shinichi Urabe to tu nguyen ngoc
  • Target version changed from OpenPNE 3.4.7 to OpenPNE 3.4.8

#11 Updated by Hiroki Mogi almost 11 years ago

テスターテストを行ないました。修正が取り消されていることを確認しました。

#12 Updated by Shinichi Urabe over 10 years ago

  • Target version changed from OpenPNE 3.4.8 to OpenPNE 3.4.9

#13 Updated by Shinichi Urabe over 10 years ago

  • Target version changed from OpenPNE 3.4.9 to OpenPNE 3.4.10

#14 Updated by tu nguyen ngoc over 10 years ago

  • Status changed from Pending Fixing(修正待ち) to Pending Review(レビュー待ち)

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

#15 Updated by Masato Nagasawa over 10 years ago

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

説明文の記入をお願いします。

#16 Updated by Shinichi Urabe over 10 years ago

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

#17 Updated by Masato Nagasawa over 10 years ago

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

レビューOKです。

#18 Updated by Yuma Sakata over 10 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

・長い文字列を入力して編集ボタンを押す
(例:abcdefghijklmnopqrstuvwxyzあいうえおかきくけこさしすせそたちつてとなにぬねのはひふへほまみむめもやゆよらりるれろわをん1234567890!”#$%&’())

・半角数字のみを入力し編集ボタンを押す
(例:1234567890)

・半角記号のみを入力し編集ボタンを押す
(例:!"#$%&'()+*`@[]{}|~^=/_<>;:)

・全角文字のみを入力し編集ボタンを押す
(例:手嶋屋手嶋屋手嶋屋)

テストした結果、上記の場合エラーメッセージが表示されないで登録可能でした。
エラーメッセージが表示されるように修正お願いします。

#19 Updated by Mutsumi Imamura over 10 years ago

坂田 雄麻 は書きました:

・長い文字列を入力して編集ボタンを押す
(例:abcdefghijklmnopqrstuvwxyzあいうえおかきくけこさしすせそたちつてとなにぬねのはひふへほまみむめもやゆよらりるれろわをん1234567890!”#$%&’())

・半角数字のみを入力し編集ボタンを押す
(例:1234567890)

・半角記号のみを入力し編集ボタンを押す
(例:!"#$%&'()+*`@[]{}|~^=/_<>;:)

・全角文字のみを入力し編集ボタンを押す
(例:手嶋屋手嶋屋手嶋屋)

テストした結果、上記の場合エラーメッセージが表示されないで登録可能でした。
エラーメッセージが表示されるように修正お願いします。

http://redmine.openpne.jp/issues/1214#note-4
のコメントにあるようにOpenPNE3.2では仕様として判断されています。
OpenPNE3.4でも仕様として問題ないと判断でよろしいでしょうか?

#20 Updated by Shinichi Urabe over 10 years ago

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

Imamura Mutsumi は書きました:

坂田 雄麻 は書きました:

・長い文字列を入力して編集ボタンを押す
(例:abcdefghijklmnopqrstuvwxyzあいうえおかきくけこさしすせそたちつてとなにぬねのはひふへほまみむめもやゆよらりるれろわをん1234567890!”#$%&’())

・半角数字のみを入力し編集ボタンを押す
(例:1234567890)

・半角記号のみを入力し編集ボタンを押す
(例:!"#$%&'()+*`@[]{}|~^=/_<>;:)

・全角文字のみを入力し編集ボタンを押す
(例:手嶋屋手嶋屋手嶋屋)

テストした結果、上記の場合エラーメッセージが表示されないで登録可能でした。
エラーメッセージが表示されるように修正お願いします。

http://redmine.openpne.jp/issues/1214#note-4
のコメントにあるようにOpenPNE3.2では仕様として判断されています。
OpenPNE3.4でも仕様として問題ないと判断でよろしいでしょうか?

#1956 (BP from #1955) で対処する想定です

#21 Updated by Masato Nagasawa over 10 years ago

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

了解しました。テスト待ちにします。

#22 Updated by Mutsumi Imamura over 10 years ago

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

テスト済みですので完了とします。

Also available in: Atom PDF