ログイン失敗するとお気に入り登録できない問題の修正

[dk-umebius]

概要(Overview・Refs Issue)

#4369 のうち、下記部分の修正

また余談ですが、ルート直下に展開しても商品アイテムの”お気に入りに追加”からログイン時に入力ミスをすると”お気に入りに追加”されず再入力後、トップページにかえされます。

方針(Policy)

AuthenticationEvents::AUTHENTICATION_FAILUREのイベントでFlashBagの'eccube.login.target.path'を引き回す

実装に関する補足(Appendix)

shopping/login.twigでログイン失敗した時にFlashBagに残ってしまうのをどうするか悩んだ結果、shopping/login.twigも修正

テンプレート修正が良くなければ、イベントリスナー内で切り分けた方が良い？

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[okazy]

以下の動作確認しました。

• マイページログイン
• お気に入りログイン
• 購入フローログイン

また同ブラウザで２画面開いて上記を組み合わせてテストしましたが問題なさそうでした。
実装、FlashBagの扱いに関しても問題ないかと思います。

テンプレート修正が良くなければ、イベントリスナー内で切り分けた方が良い？

こちら私がイメージできていません。
「テンプレート修正」とは何をさしているんでしょうか？

[okazy]

プルリクの取り込みありがとうございます！

[kiy0taka]

#4500 がシンプルにこの問題に解決しているので、4500を取り込みこちらはクローズさせてもらいます。
FlashBagに残る問題はありますが、影響はないと考えます。

[kiy0taka]

問題があれば再オープンします。