[Koha-bugs] [Bug 13330] All overdue items with fines returned via checkouts table will have fines forgiven!

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13330

Liz Rea <liz at catalyst.net.nz> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #33875|0                           |1
        is obsolete|                            |

--- Comment #3 from Liz Rea <liz at catalyst.net.nz> ---
Created attachment 33881
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=33881&action=edit
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 at catalyst.net.nz>
Verified bug, fix corrects the problem.

-- 
You are receiving this mail because:
You are watching all bug changes.