商品別税率機能の不具合 about ec-cube2

[nomoto-neo]

2.13で、商品別税率が商品一覧と商品詳細に反映されない件ですが、軽減税率対応のためにも対処しなければならない問題でしたので独自に修正しました。

少しでも当プロジェクトに貢献したいのですが、Githubの扱いが不慣れでプロジェクトへの反映方法が分からないため、間違ったら困るのでこちらにdiffを貼り付けさせてください。

OPTION_PRODUCT_TAX_RULE = 0 の場合は商品個別設定値を無視して基本税率を適用します。
TAX_RULE_PRIORITY の設定値は2.13でどのように機能しているのか不明だったため考慮していません。
product_list_and_detail_taxrate_bug_fix.diff.txt

→ 追記）商品規格ごとの税率設定を削除 #301 の対応の方がスマートで、そちらが正採用されましたのでそちらの投入をお勧めします。上記パッチは参考程度にしてください。

[seasoftjapan]

最近、商品別税率機能を検証しましたが、色々と問題ありますね・・・

@nomoto-neo さんの対応された部分、私も SQL での処理が妥当かもと思っていました。

あと、商品登録で、税率未設定を登録できなかったり、端数処理まで登録されるのは、どうなんだろと疑問に思いました。(そもそも、dtb_tax_rule.tax_rate は良くない印象です。2.12 に dtb_products_class.tax_rate を付け加えた経験がありますが、そっちの方が勝手が良かったです。)
CSV対応できていないのも、実運用を考えると痛いですね・・・ (これも、dtb_products_class.tax_rate なら、対応が簡単なのに…)

[nanasess]

@nobuhiko SC_Product::setIncTaxToProduct() で product_class_id を渡せないかと修正を試みてますー

[nomoto-neo]

product_class_id単位の情報なのにdtb_tax_ruleに値を持たせている事で、わざわざJOINしたりサブクエリを使わないといけないのが、確かに修正していて、修正を尚更難しくしている印象でした。

スマートにdtb_products_classにtax_rateを持たせるべきだと私も考えましたが、管理画面の修正も入ると2.13からの修正箇所がさらに大きくなりそうで、軽減税率開始2ヶ月を切った現時点でそこまでの修正コストは掛けれなかったです。

[nanasess]

カテゴリや商品単位で税率を指定できるよう、このような実装になってるんですよね。
また、この機能を開発していた当初は国際化版のバージョンもあった関係で、国や都道府県ごとに税率設定可能なデータ構造になってます

[nobuhiko]

@nanasess さん、このチケット対応します？

[nobuhiko]

関連事項としては、

• 納品書
  の「区分記載請求書等」対応が必要になる

[nanasess]

商品規格ごとの税率設定を削除することで対応 #301