EC-CUBE / EC-CUBE/ec-cube

PurchaseFlowの税計算タイミングの改善

Open
#4,532 2 comments 0 reactions 0 assignees View on GitHub
improvement
Dominant language
PHP
Stars
788
Forks
719
Avg merge
4d 4h
Merged PRs (30d)
39

Description

## 概要(Overview)

PurchaseFlowのshoppingフローにおいて、ItemHolderPreprocessor内の最初と最後でTaxProcessorが実行されるようになっている。

https://github.com/EC-CUBE/ec-cube/blob/538c144374506c056f7cf96353b080cad53be22f/app/config/eccube/packages/purchaseflow.yaml#L73-L89

### Processor追加時の考慮
アノテーションでProcessorを追加した場合、実行は最後に追加される。
TaxProcessor実行後に追加されるため、アノテーションでItemHolderPreprocessorを追加した場合にはもう一度TaxProcessorを実行してやる必要がある。

### DiscountProcessorの考慮
DiscountProcessorで追加された明細にはTaxProcessorが適用されない。
またTaxProcessorはDiscountProcessorインタフェースを実装していないため、DiscountProcessorで実行することができない。
現状ではDiscountProcessor内で税の計算をする必要がある。
TaxProcessorはDiscountProcessorの最後でも実行したほうがよい。

## 期待する内容(Expect) or 要望 (Requirement)

アノテーションでのProcessor追加時にも正しく税計算がされる。

## 再現手順(Procedure)

アノテーションで税の再計算が必要な明細の変更または追加。

### 環境 (environment)
+ EC-CUBE: 4.0.3

## 関連情報 (Ref)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.