]> git.koha-community.org Git - koha.git/commit
Bug 29754: Don't include patron when fetching guarantees fines
authorNick Clemens <nick@bywatersolutions.com>
Wed, 22 Dec 2021 15:25:44 +0000 (15:25 +0000)
committerVictor Grousset/tuxayo <victor@tuxayo.net>
Fri, 18 Mar 2022 00:00:21 +0000 (01:00 +0100)
commit727376cdca32627ba0fb43fd2c5a885639855dd2
tree1352e7dfe7a440a81930e0f31ba3588f37674cc0
parent487eddaf1e46c1718b9b3a1b426a3db132e8f87d
Bug 29754: Don't include patron when fetching guarantees fines

This patch removes the flag to fetch the patrons own debts as we have already
calculated them

This test plan highlights another issue - the value of NoIssuesChargeGuarantorsWithGuarantees
is not checked in SIP code - we merely get the total owed including guarantees and compares to
noissuescharge
See bug 29755

To test:
 1 - Apply first patch only
 2 - prove t/db_dependent/SIP/Patron.t
 3 - It fails
 4 - Enable NoIssuesChargeGuarantorsWithGuarantees, you can set to 1
 5 - Create/find a patron
 6 - Add a $10 charge to parons account
 7 - Add a guarantee to patron
 8 - Add a $5 charge to guarantee
 9 - Set noissuescharge to 15.01
10 - perl misc/sip_cli_emulator.pl -a localhost -p 6001 -su term1 -sp term1 -l CPL -m patron_information --patron BARCODE
11 - Note the 64 message starts with Y's that mean patron is blocked
12 - Patron shouldn't be blocked as under 15.01
13 - Apply patch
14 - Restart all
15 - repeat 10
16 - Patron is no longer blocked

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
(cherry picked from commit fe59e356a3b2ea32b333d6c1d332468db61feeec)

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
(cherry picked from commit c5084b48f038b5c189614c65ab0741d10f1a2f4a)
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
C4/SIP/ILS/Patron.pm