Bug 17668: Fix typo in parcel.pl - listinct vs listincgst
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 22 Nov 2016 15:28:04 +0000 (15:28 +0000)
committerKatrin Fischer <katrin.fischer.83@web.de>
Mon, 12 Dec 2016 22:11:05 +0000 (23:11 +0100)
commitc6b0b9cf1b46027e44cee0730befa8d8ac40efbd
tree82a134b6c79c948a211028ea02e4ed1b6a141b99
parenta5ce343c9d7d8790a8dbd98b3833ea6cf155c180
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>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit 3e112f3837346b500ce863a248920d3207e27131)
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
acqui/parcel.pl