Project

General

Profile

Backport(バックポート) #2374

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

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

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

100%


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(バグ) #2373: メール通知テンプレート設定にてエラーのアラートが表示され続ける Fixed(完了) 2011-08-16

Associated revisions

Revision 98942fd9 (diff)
Added by Maki Takahashi over 8 years ago

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

History

#1 Updated by Maki Takahashi over 8 years ago

  • Status changed from New(新規) to Accepted(着手)

#2 Updated by Anonymous over 8 years ago

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

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

#3 Updated by Kousuke Ebihara over 8 years ago

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

#4 Updated by Fumie Toyooka over 8 years ago

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

テストOKです。

Also available in: Atom PDF