- Issue created by @TomTech
- Status changed to Needs review
about 2 years ago 3:21am 4 April 2023 - 🇺🇸United States TomTech
This patch clones the Order PriceSplitter class and interface, and modifies it to support InvoiceInterface and InvoiceItemInterface.
It also replicates the PriceSplitterTest class to provide test coverage.
(It also adds a .gitignore file, also copied from commerce core.)
- 🇺🇸United States TomTech
Try that one more time with the complete diff on the patch. :)
- 🇺🇸United States TomTech
There is actually an issue in the order price splitter when splitting a negative fixed amount, that was carried over into this implementation.
This patch addresses that issue, along with adding an additional test case to test that scenario.
I'll also be submitting a similar patch in commerce core to keep them in sync.
- Merge request !11Issue #3351994 by tomtech: Invoice price splitter needed → (Merged) created by TomTech
- 🇺🇸United States TomTech
re-rolled and created as an MR. previous patches hidden.
- First commit to issue fork.
- 2305a5aa committed on 8.x-2.x
Issue #3351994 by tomtech: Invoice price splitter needed
- 2305a5aa committed on 8.x-2.x
Automatically closed - issue fixed for 2 weeks with no activity.