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

税率設定:商品CSVの入出力で、商品別税率を登録できるようにする #1741 #4281

Merged
merged 5 commits into from Aug 24, 2019

Conversation

@okazy
Copy link
Contributor

commented Aug 22, 2019

概要(Overview・Refs Issue)

税率設定:商品CSVの入出力で、商品別税率を登録できるようにする
#1741 の対応

  • 商品CSV出力で商品別税率を出力可能に
    • dtb_csvの初期データに税率を追加
    • マイグレーションを追加
  • 商品CSV入力で商品別税率を入力可能に
    • 規格なし商品新規登録
    • 規格なし商品更新
    • 規格新規登録( #4282 の問題は未解決)
    • 規格更新
    • #4277 とのコンフリクト解消
  • UnitTest

方針(Policy)

  • 個別税率設定が有効な場合にのみ個別税率を更新する(商品ごとの送料設定と同様の条件)
  • 商品別税率は dtb_product_class ではなく dtb_tax_rule に保存される
    • 商品別税率を新しく設定する場合は dtb_tax_rule のレコードを新しく追加
    • 商品別税率の設定を解除する場合は dtb_tax_rule のレコードを削除(物理削除)

実装に関する補足(Appendix)

テスト(Test)

概要のチェック項目に従ってテストを実施

相談(Discussion)

商品CSVへの税率の出力は dtb_csv へレコードを追加することで実施している。
インストール時の初期データに追加したのと、マイグレーションで追加する処理を追加している。
問題がないか念のために確認をお願いします。

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

CSV関連の仕様変更あり

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

レビュワー確認項目

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

@okazy okazy changed the base branch from 4.0 to dev/reduced_tax_rate Aug 22, 2019

@okazy okazy added this to the 4.0.3 milestone Aug 22, 2019

@okazy okazy changed the title [WIP]税率設定:商品CSVの入出力で、税率を登録できるようにする #1741 [WIP]税率設定:商品CSVの入出力で、商品別税率を登録できるようにする #1741 Aug 22, 2019

@okazy okazy changed the title [WIP]税率設定:商品CSVの入出力で、商品別税率を登録できるようにする #1741 税率設定:商品CSVの入出力で、商品別税率を登録できるようにする #1741 Aug 22, 2019

@okazy okazy force-pushed the okazy:dev/1741 branch 2 times, most recently from 67b52d3 to 3e3065f Aug 22, 2019

@okazy okazy force-pushed the okazy:dev/1741 branch from 3e3065f to b7ca26b Aug 22, 2019

@chihiro-adachi chihiro-adachi merged commit 4c17dd3 into EC-CUBE:dev/reduced_tax_rate Aug 24, 2019

2 checks passed

Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
return;
}
$this->addSql("INSERT INTO dtb_csv (id, csv_type_id, creator_id, entity_name, field_name, reference_field_name, disp_name, sort_no, enabled, create_date, update_date, discriminator_type) VALUES (204, 1, null , 'Eccube\\\\Entity\\\\ProductClass', 'TaxRule', 'tax_rate', '税率', 31, 0, CURRENT_TIMESTAMP, CURRENT_TIMESTAMP,'csv')");

This comment has been minimized.

Copy link
@kiy0taka

kiy0taka Sep 5, 2019

Contributor

プラグインからレコードが追加されてる場合があるので、id=204固定はまずいです。

@kiy0taka kiy0taka referenced this pull request Sep 5, 2019
11 of 11 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.