Bug 6490 - Lost and paid not updated when book is checked out without check-in
Summary: Lost and paid not updated when book is checked out without check-in
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: 3.6
Hardware: Macintosh Linux
: P5 - low major (vote)
Assignee: Kyle M Hall
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-09 18:21 UTC by Katie
Modified: 2013-12-05 19:59 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Add call to _FixAccountForLostAndReturned from AddIssue (1.00 KB, patch)
2012-02-02 15:22 UTC, Kyle M Hall
Details | Diff | Splinter Review
Bug 6490 - Lost and paid not updated when book is checked out. (1.35 KB, patch)
2012-02-02 17:58 UTC, Liz Rea
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katie 2011-06-09 18:21:11 UTC
After a book is marked lost and is billed to the patrons account and the book is seen again as a new checkout (it was not checked in) it does not take the bill off the patrons account.  However when you check in the book it does take the bill off the patron's account.  Whenever a book whose status is "Lost" is seen in the system again either through checkout, holds, ect... it should update the last borrowers account and book status.
Comment 1 Liz Rea 2011-09-23 02:11:26 UTC
Confirmed that even after 5533 is applied, this is still broken.
Comment 2 Kyle M Hall 2012-02-02 15:22:24 UTC Comment hidden (obsolete)
Comment 3 Liz Rea 2012-02-02 17:58:47 UTC
Created attachment 7422 [details] [review]
Bug 6490 - Lost and paid not updated when book is checked out.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>

Tested:

item checked out and marked lost - item is removed from borrower's checkouts and charged to patron
Checking out the item (without checking it in first) removed the fines and issued the item normally.

Passed prove t xt t/db_dependent (results congruent with current master failures)
Comment 4 Paul Poulain 2012-02-03 16:51:08 UTC
QA comment: tiny patch, respect coding guidelines
Comment 5 Jared Camins-Esakov 2012-05-23 12:59:11 UTC
This fix was included in the 3.6.x branch prior to 3.6.4.