商品一覧及び商品詳細で個別税率が取得できないのを修正

[nanasess]

概要(Overview・Refs Issue)

• 商品一覧及び商品詳細で個別税率が取得できないのを修正 ec-cube#4310 からの移行
• Fixes 個別税率が適用されない #46
• TaxRuleEventSubscriber で ProductClass::getProduct() が null を返してしまう場合の対策
• #2158 のデグレ修正

方針(Policy)

• ProductClass !== null かつ Product === null の場合は Product を取得し直すよう修正

実装に関する補足(Appendix)

• ProductClass を refresh することで Product が lazy loading の対象となる

テスト（Test)

ユニットテストを追加

相談（Discussion）

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

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

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[ryo-endo]

@nanasess 遅くなりましたが取り込ませていただきたい...のですがscrutinizerさんが警告を出しているので、またお手すきのときにご確認お願いします。

[nanasess]

@ryo-endo
\Eccube\Entity\ProductClass ではなく、 use を使ってねという警告ですが、同クラスの他の既存ロジックも \Eccube\Entity\ClassName でやっています。
このままでも問題ないと思いますがいかがでしょうか？