メール設定・CSV出力項目設定：ブラウザバック不具合の修正

[riki-develop]

概要(Overview・Refs Issue)

メールテンプレート編集画面でブラウザバックをすると、
テンプレート名と本文が食い違った表示になる #5670
↓
複数回select要素の変更を行いブラウザバックすると表示キャッシュが残り、
select部分のみ前ページの文字列が保持されてしまう

方針(Policy)

ブラウザの「戻る」イベントが発生した場合のみ、
option要素のselected属性を再設定し表示を更新

実装に関する補足(Appendix)

同様の不具合が発生するページ【CSV出力項目設定】も修正

※補足
EC-CUBE4.2以前のversionでも同じ不具合を確認

テスト（Test)

▼下記URLにて動作確認
/admin/setting/shop/mail
/admin/setting/shop/csv

▼開発環境（M1 Mac：Docker）

EC-CUBE：4.2.0-beta2
DB：MySQL 5.7.39
Apache/2.4.54 (Debian)
PHP：7.4.30

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

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

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか
  • 権限を超えた操作が可能にならないか
  • 不要なファイルアップロードがないか
  • 外部へ公開されるファイルや機能の追加ではないか
  • テンプレートでのエスケープ漏れがないか

[chihiro-adachi]

@riki-develop
safari/firefoxでは修正確認できましたが、chromeでは改善されませんでした。
※OSはMac Montereyです

どうもwindow.performance.navigation.typeがブラウザバック時でも0が返る挙動になるようです。

[chihiro-adachi]

@riki-develop
少し手元で試してみただけなのですが、以下のコードだけでも良いのかなと思いましたがいかがでしょうか。

            $(window).on('pageshow', function() {
                var tmp_select =  $('#mail_template[name="mail[template]"] option[selected]').val();
                $('#mail_template[name="mail[template]"]').val(tmp_select);
            });

[riki-develop]

@chihiro-adachi
ご確認ありがとうございます。
共有頂いたコードで問題ない事を確認しました。

▼下記、不要な処理を削除
window.performance.navigation.type
... removeAttr('selected');

そもそもwindow.performance.navigation.typeは非推奨でした、失礼しました。
https://developer.mozilla.org/ja/docs/Web/API/PerformanceNavigation/type

[chihiro-adachi]

@riki-develop
ありがとうございます。修正確認しました

[codecov-commenter]

Codecov Report

Merging #5700 (942ee07) into 4.2 (0c42641) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                4.2    #5700      +/-   ##
============================================
- Coverage     78.84%   78.84%   -0.01%     
  Complexity     6273     6273              
============================================
  Files           469      469              
  Lines         21058    21058              
============================================
- Hits          16604    16603       -1     
- Misses         4454     4455       +1     

Flag	Coverage Δ
E2E	64.82% <ø> (-0.02%)	⬇️
Unit	77.59% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...ontroller/Admin/Product/ProductClassController.php	90.28% <0.00%> (-0.58%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.