Skip to content

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

Closed
nanasess wants to merge 2 commits intoEC-CUBE:3.0from
nanasess:fix-taxrule
Closed

商品一覧及び商品詳細で個別税率が取得できないのを修正#4310
nanasess wants to merge 2 commits intoEC-CUBE:3.0from
nanasess:fix-taxrule

Conversation

@nanasess
Copy link
Contributor

@nanasess nanasess commented Sep 6, 2019

概要(Overview・Refs Issue)

方針(Policy)

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

実装に関する補足(Appendix)

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

テスト(Test)

ユニットテストを追加

相談(Discussion)

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

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

レビュワー確認項目

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

TaxRuleEventSubscriber で ProductClass::getProduct() が null を返してしまう場合の対策
@chihiro-adachi chihiro-adachi added this to the 3.0.19 milestone Sep 6, 2019
@nanasess
Copy link
Contributor Author

nanasess commented Dec 4, 2019

EC-CUBE/ec-cube3#73 に移行しました

@nanasess nanasess closed this Dec 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants