]> git.koha-community.org Git - koha.git/commit
Bug 13841 - Returns via backdating or SIP2 can create new fines with an amount outsta...
authorKyle M Hall <kyle@bywatersolutions.com>
Mon, 16 Mar 2015 14:16:40 +0000 (10:16 -0400)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Tue, 31 Mar 2015 14:36:07 +0000 (11:36 -0300)
commitfaab00b88a155bdd7d5de35ef9226e134ce4b35b
treed32a21b72b8509d7e57f0cfbb8e2c77b58fe108d
parent589fd84107ed76ed50397f63d4ca809468f57ffe
Bug 13841 - Returns via backdating or SIP2 can create new fines with an amount outstanding of 0

Due to the way AddReturn and UpdateFine work, if you pass in a return
date to AddReturn, and that item has a calculated fine of 0, Koha will
insert the 0 amount fine into accountlines.

This is good behavior when we want to update an existing fine, but bad
behavior if there was no fine to begin with!

Test Plan:
1) Apply the unit test patch
2) prove t/db_dependent/Circulation.t
3) Note the test fails
4) Apply this patch
5) prove t/db_dependent/Circulation.t
6) Note the test passes

Signed-off-by: Jen DeMuth <jdemuth@roseville.ca.us>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Confirmed the problem in the staff interface and that the patch
fixes it. Also tested that fines still get calculated.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
C4/Overdues.pm