[4.1.x, 4.2.x] 値をスペースで更新してから更新をキャンセルすると、元の値が消える不具合の修正

[maianhtu010596]

概要(Overview・Refs Issue)

以下のissueの対応です。
#5792 [4.1.x, 4.2.x] 値をスペースで更新してから更新をキャンセルすると、元の値が消えます

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

この問題を修正し、次のような機能テストを行いました。

• /product/class_name (規格管理)
  
  

• /product/category (カテゴリ管理)
  
  

• /product/tag (タグ管理)
  
  

• /setting/shop/delivery/{id}/edit (配送方法設定)
  
  

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5828 (11136df) into 4.2 (4c27230) will decrease coverage by 0.10%.
The diff coverage is 93.75%.

@@             Coverage Diff              @@
##                4.2    #5828      +/-   ##
============================================
- Coverage     78.86%   78.76%   -0.11%     
- Complexity     6274     6289      +15     
============================================
  Files           469      470       +1     
  Lines         21076    21104      +28     
============================================
  Hits          16622    16622              
- Misses         4454     4482      +28     

Flag	Coverage Δ
E2E	64.79% <62.50%> (-0.08%)	⬇️
Unit	77.78% <93.75%> (+0.17%)	⬆️

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

Impacted Files	Coverage Δ
src/Eccube/Form/Type/Admin/NewsType.php	91.30% <ø> (ø)
src/Eccube/Form/Type/Admin/TradeLawType.php	100.00% <ø> (ø)
src/Eccube/Form/Type/Admin/CalendarType.php	88.88% <85.71%> (-1.12%)	⬇️
...Eccube/Controller/Admin/Content/FileController.php	84.70% <100.00%> (-0.28%)	⬇️
...ube/Controller/Admin/Product/ProductController.php	79.63% <100.00%> (+1.71%)	⬆️
...ontroller/Admin/Setting/Shop/PaymentController.php	55.30% <100.00%> (+2.27%)	⬆️
...e/Form/Extension/HTMLPurifierTextTypeExtension.php	84.61% <100.00%> (ø)
src/Eccube/Form/Type/Admin/ProductType.php	98.11% <100.00%> (+0.11%)	⬆️
...e/Validator/EmailValidator/NoRFCEmailValidator.php	29.41% <0.00%> (-70.59%)	⬇️
src/Eccube/Util/FilesystemUtil.php	75.00% <0.00%> (-25.00%)	⬇️
... and 82 more

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

[chihiro-adachi]

@maianhtu010596
こちらの修正に合わせていただくことはできますか？
#5517

[maianhtu010596]

@maianhtu010596 こちらの修正に合わせていただくことはできますか？ #5517

@chihiro-adachi
issues/5492 の pull/5517 の修正を更新しました
ご確認お願い致します

この問題を修正し、次のような機能テストを行いました。
Step 1.

Step 2.

Step 3.

[maianhtu010596]

@chihiro-adachi
問題を確認したところ、休日フォームに制限を超える日付を入力すると問題が発生することがわかりました。
これは解決すべきだと思ったので、修正コードを追加しました。
ご確認お願い致します

この問題を修正し、次のような機能テストを行いました。
Step 1. 制限を超えた日付を入力すると、エラーが表示されます。

Step 2. キャンセルボタンを押します

Step 3. 編集をもう一度押すと、元の日付が表示されます。

[chihiro-adachi]

@maianhtu010596
ごめんなさい。

#5492 の #5517 の修正を更新しました

#5517 の修正を採用したいです。
本PullRequestの修正を、#5517と同様の修正に合わせていただくことはできますか？

[maianhtu010596]

@chihiro-adachi

問題 #5571 に類似した問題 #5792 の修正を更新しました。
ご確認お願い致します

この問題を修正し、次のような機能テストを行いました。

Test 1.
/product/class_name (規格管理)
/product/category (カテゴリ管理)
/product/tag (タグ管理)
/setting/shop/delivery/{id}/edit (配送方法設定)
/product/class_category/{} (規格分類管理)
/setting/shop/calendar (定休日カレンダー設定)
キャンセル ボタンをクリックすると、元のデータが表示された状態でページが更新されます。

Test 2.
/setting/shop/calendar (定休日カレンダー設定)
制限を超える日付を入力すると、エラー メッセージが表示されます。

[chihiro-adachi]

@maianhtu010596
対応ありがとうございます。
以下2点修正しています。

• 配送方法設定は、他の項目もリセットされるため対応なし
• window.location.hrefでなくルーティングからurlを生成するように修正