プロジェクト

全般

プロフィール

Backport(バックポート) #2374

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

Maki Takahashi12年以上前に追加. 12年以上前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2011-08-16
期日:
進捗率:

100%


説明

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にする


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2373: メール通知テンプレート設定にてエラーのアラートが表示され続ける Fixed(完了) 2011-08-16

関係しているリビジョン

リビジョン 98942fd9 (差分)
Maki Takahashi12年以上前に追加

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

履歴

#1 Maki Takahashi12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#2 匿名ユーザー が12年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#3 Kousuke Ebihara12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#4 Fumie Toyooka12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

他の形式にエクスポート: Atom PDF