Symfony/HttpFoundation

[kiy0taka]

概要(Overview・Refs Issue)

#33

方針(Policy)

実装に関する補足(Appendix)

• Controllerの戻り値は Symfony\Component\HttpFoundation\Responseである必要があるので、Eccube\Http\ResponseListener で変換しています。
• HttpExceptionはアダプター (Eccube\Http\Exception\*Exception)を作って Eccube\Http\Exception\ExceptionListener で HttpFoundationのExceptionに変換しています。

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #47 (6ef4be9) into main (60179f7) will increase coverage by 0.03%.
The diff coverage is 83.33%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@             Coverage Diff              @@
##               main      #47      +/-   ##
============================================
+ Coverage     79.60%   79.64%   +0.03%     
- Complexity     7052     7164     +112     
============================================
  Files           565      595      +30     
  Lines         27798    28043     +245     
============================================
+ Hits          22129    22335     +206     
- Misses         5669     5708      +39     

Flag	Coverage Δ
E2E	69.08% <78.29%> (+0.10%)	⬆️
Unit	77.24% <82.55%> (+0.06%)	⬆️

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

Impacted Files	Coverage Δ
...n/Eccube/Controller/AbstractShoppingController.php	100.00% <ø> (ø)
...e/Controller/Admin/AbstractCsvImportController.php	96.15% <ø> (ø)
...cation/Eccube/Controller/Admin/AdminController.php	97.93% <ø> (ø)
...ccube/Controller/Admin/Content/BlockController.php	97.95% <ø> (ø)
...ccube/Controller/Admin/Content/CacheController.php	100.00% <ø> (ø)
.../Eccube/Controller/Admin/Content/CssController.php	100.00% <ø> (ø)
...n/Eccube/Controller/Admin/Content/JsController.php	100.00% <ø> (ø)
...cube/Controller/Admin/Content/LayoutController.php	89.24% <ø> (ø)
...Controller/Admin/Content/MaintenanceController.php	97.05% <ø> (ø)
...Eccube/Controller/Admin/Content/NewsController.php	92.18% <ø> (ø)
... and 109 more

... and 1 file with indirect coverage changes