Created attachment 57713 [details] [review] Bug 17668: Fix typo in parcel.pl - listinct vs listincgst This typo has been introduced by commit eed14b080db8691f371b4c19f1a190b87fcc396a Bug 13001: Refactor VAT and price calculation - parcel page So before the tax rewrite (13321, 13323). It sounded weird to me that testers did not catch the bug on this page. To understand the logic: Conditions are listincgst, invoiceincgst Conditions | Before this patch | If we fix the typo | After this patch 0 0 | excluded | excluded | excluded 0 1 | included | included | included 1 0 | excluded | excluded | excluded 1 1 | included | included | included Test plan: Create 4 vendors with the difference combinations Create a basket, add an order (with a tax) and receive it Confirm that the different values displayed on the parcel page are correct
Created attachment 57790 [details] [review] [SIGNED-OFF] Bug 17668: Fix typo in parcel.pl - listinct vs listincgst This typo has been introduced by commit eed14b080db8691f371b4c19f1a190b87fcc396a Bug 13001: Refactor VAT and price calculation - parcel page So before the tax rewrite (13321, 13323). It sounded weird to me that testers did not catch the bug on this page. To understand the logic: Conditions are listincgst, invoiceincgst Conditions | Before this patch | If we fix the typo | After this patch 0 0 | excluded | excluded | excluded 0 1 | included | included | included 1 0 | excluded | excluded | excluded 1 1 | included | included | included Test plan: Create 4 vendors with the difference combinations Create a basket, add an order (with a tax) and receive it Confirm that the different values displayed on the parcel page are correct Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 58011 [details] [review] Bug 17668: Fix typo in parcel.pl - listinct vs listincgst This typo has been introduced by commit eed14b080db8691f371b4c19f1a190b87fcc396a Bug 13001: Refactor VAT and price calculation - parcel page So before the tax rewrite (13321, 13323). It sounded weird to me that testers did not catch the bug on this page. To understand the logic: Conditions are listincgst, invoiceincgst Conditions | Before this patch | If we fix the typo | After this patch 0 0 | excluded | excluded | excluded 0 1 | included | included | included 1 0 | excluded | excluded | excluded 1 1 | included | included | included Test plan: Create 4 vendors with the difference combinations Create a basket, add an order (with a tax) and receive it Confirm that the different values displayed on the parcel page are correct Signed-off-by: Josef Moravec <josef.moravec@gmail.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Pushed to master for 17.05, thanks Jonathan!
This patch has been pushed to 16.11.x, will be in 16.11.01.