操作
Backport(バックポート) #2375
完了メール通知テンプレート設定にてエラーのアラートが表示され続ける
開始日:
2011-08-16
期日:
2011-10-05
進捗率:
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にする
Maki Takahashi さんが約13年前に更新
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 078e0bee02d684dbaddce40e1fce412803d082b7 で適用されました。
Shinichi Urabe さんが約13年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
修正内容確認しました
操作