オプション引数が必須引数より先に来ているコードを修正

[takeuji]

概要(Overview・Refs Issue)

オプション引数が必須引数より先に来ているコードを修正しました

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5691 (e7731e8) into 4.2 (16561c7) will decrease coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                4.2    #5691      +/-   ##
============================================
- Coverage     78.82%   78.75%   -0.08%     
- Complexity     6260     6269       +9     
============================================
  Files           469      468       -1     
  Lines         21010    21024      +14     
============================================
- Hits          16561    16557       -4     
- Misses         4449     4467      +18     

Flag	Coverage Δ
E2E	64.71% <100.00%> (-0.07%)	⬇️
Unit	77.49% <100.00%> (-0.04%)	⬇️

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

Impacted Files	Coverage Δ
...e/Controller/Admin/Setting/Shop/MailController.php	86.20% <100.00%> (ø)
src/Eccube/Service/MailService.php	82.26% <0.00%> (-8.08%)	⬇️
...ube/Controller/Admin/Order/CsvImportController.php	85.55% <0.00%> (-1.81%)	⬇️
...ype/Admin/OrderItemForShippingRegistrationType.php
...Eccube/Controller/Admin/Content/FileController.php	84.86% <0.00%> (+0.06%)	⬆️
...be/Service/PurchaseFlow/Processor/TaxProcessor.php	84.21% <0.00%> (+1.75%)	⬆️
.../Service/PurchaseFlow/Processor/PointProcessor.php	95.83% <0.00%> (+1.95%)	⬆️

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