Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

商品別税率設定の修正 #4331

Merged
merged 7 commits into from
Sep 25, 2019
Merged

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Sep 19, 2019

概要(Overview・Refs Issue)

#4330 の修正

方針(Policy)

  • 商品別税率が設定されている場合は、常に最優先するよう修正

実装に関する補足(Appendix)

  • Pref や Country が設定されている場合でも、商品別税率を優先する
  • 商品別税率設定は同じ商品、商品規格で重複して登録されてないことを前提とする
  • TaxRule の proxy クラスが生成されている場合は、再生成が必要

テスト(Test)

  • テストケースを追加

相談(Discussion)

  • この修正を適用すると、商品別税率設定では適用日を考慮しなくなるため、事前に設定している場合は注意が必要
  • eccube_tax_rule_priority が意味を無さなくなる

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

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

レビュワー確認項目

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

@okazy okazy added the bug:High label Sep 20, 2019
@okazy okazy added this to the 4.0.4 milestone Sep 20, 2019
@nanasess nanasess changed the title [WIP] 商品別税率設定の修正 商品別税率設定の修正 Sep 20, 2019
@chihiro-adachi
Copy link
Contributor

@nanasess
商品別税率が優先されることを確認しました。修正ありがとうございます。

@chihiro-adachi
Copy link
Contributor

pgsql/mysqlともに動作確認しています。マージさせていただきます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:外部仕様 外部仕様の変更や追加 bug:High
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants