CSVを出力するとシステムエラーになる不具合の修正

[chihiro-adachi]

概要(Overview・Refs Issue)

#5754 の修正

クエリビルダ構築時にセッションから検索条件を取得しているが、すでにCSVヘッダが出力されているため、Failed to start the session because headers have already been sentのエラーが発生していた。

クエリビルダの構築とヘッダ出力の順序を逆にすることで修正しています。

テスト（Test)

CSVが正常に出力されることを確認

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5892 (a471ef5) into 4.2 (9f6a2ab) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##                4.2    #5892   +/-   ##
=========================================
  Coverage     78.87%   78.87%           
  Complexity     6346     6346           
=========================================
  Files           473      473           
  Lines         21327    21327           
=========================================
  Hits          16822    16822           
  Misses         4505     4505           

Flag	Coverage Δ
E2E	64.83% <100.00%> (-0.02%)	⬇️
Unit	77.82% <100.00%> (ø)

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

Impacted Files	Coverage Δ
...e/Controller/Admin/Customer/CustomerController.php	83.73% <100.00%> (ø)
.../Eccube/Controller/Admin/Order/OrderController.php	80.55% <100.00%> (ø)
...ube/Controller/Admin/Product/ProductController.php	79.63% <100.00%> (ø)
src/Eccube/Entity/TaxRule.php	85.18% <0.00%> (-1.24%)	⬇️
...be/Service/PurchaseFlow/Processor/TaxProcessor.php	84.21% <0.00%> (+1.75%)	⬆️

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