Project

General

Profile

Bug(バグ) #2373

メール通知テンプレート設定にてエラーのアラートが表示され続ける

Added by Maki Takahashi over 8 years ago. Updated over 4 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2011-08-16
Due date:
% Done:

100%

3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

Description

Overview (現象)

メール通知テンプレート設定で、任意のメール通知テンプレートを編集時にエラーになった場合(例:CSRFエラー)
「_csrf_token [Required.]」などのアラートが表示されるが、
その後、他の画面に遷移した場合アラートが消えずに残ってしまう。
(さらに他の画面に遷移した場合にアラートが消える)

再現バージョン

  • OpenPNE3.7 masterブランチ
  • OpenPNE3.6beta13-dev
  • OpenPNE3.4.15-dev

再現手順

1.管理画面の「メール設定」>「メール通知テンプレート設定」(/mail/edit)にアクセスする
2.任意のテンプレートを選択し、
Firebugを利用してフォームのhidden属性の項目であるnotification_mail_translation[_csrf_token]のvalueを削除した状態で「確定」ボタンを押す
3.「_csrf_token [Required.]」表示されることを確認する
4. 3の状態で、任意のリンクをクリックする(例:「管理画面トップ」)
4.「_csrf_token [Required.]」の表示がされたままになる

Causes (原因)

#2353 と同様に、apps/pc_backend/modules/mail/actions/actions.class.phpのexecuteTemplate()内で
$this->getUser()->setFlash()を利用してセットしているが
上記メソッドは次のアクションまでセットされたメッセージを保持するため
別のページに行った場合でも
メッセージが表示され続けてしまっている

Way to fix (修正内容)

$this->getUser()->setFlash()の第3引数($persist)をfalseにする


Related issues

Related to OpenPNE 3 - Bug(バグ) #2353: ログインしてくださいというアラートが表示され続ける Fixed(完了) 2011-08-16
Related to OpenPNE 3 - Backport(バックポート) #2374: メール通知テンプレート設定にてエラーのアラートが表示され続ける Fixed(完了) 2011-08-16
Related to OpenPNE 3 - Backport(バックポート) #2375: メール通知テンプレート設定にてエラーのアラートが表示され続ける Fixed(完了) 2011-08-16 2011-10-05

Associated revisions

Revision 0283aeb5 (diff)
Added by Maki Takahashi over 8 years ago

(fixes #2373) chenged sefFlash() persist param to false in case of invalid in mail/edit

History

#1 Updated by Anonymous over 8 years ago

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

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

#2 Updated by Kousuke Ebihara over 8 years ago

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

#3 Updated by Yuma Sakata about 8 years ago

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

テストOKです。

#4 Updated by kaoru n over 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF