Bug 13330 - All overdue items with fines returned via checkouts table will have fines...
authorKyle M Hall <kyle@bywatersolutions.com>
Mon, 24 Nov 2014 15:18:50 +0000 (10:18 -0500)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Tue, 25 Nov 2014 18:25:14 +0000 (15:25 -0300)
commit6997276e514a940439906ed986bd4c93d1286083
tree1fb1dbf6e9f57301a5369fc161bf89c235bb19b0
parent33f53e4dfd79f122fac0d4daf3796ce163564580
Bug 13330 - All overdue items with fines returned via checkouts table will have fines forgiven!

Due to a disconnect between how booleans are handled in Perl vs JSON,
all checkins done from the checkouts table will have fines forgiven!
This is because the parameters sent to svc/checkin are coded in JSON
and the JSON false is treated in Perl as the string 'false' which
evaluates to true!

Test Plan:
1) Find a patron with an overdue item that has a fine
2) Return the item via the checkouts table on circulation.pl or moremember.pl
3) Note the fine is forgiven
4) Apply this patch
5) Repeat steps 1-2
6) Note the fine is no longer forgiven

Signed-off-by: Liz Rea <liz@catalyst.net.nz>
Verified bug, fix corrects the problem.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Also tested waiving fines form the check-in page still works.
Passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
svc/checkin