Bug 7526

Summary: longoverdue.pl leaves items marked as lost as still checked out to patron
Product: Koha Reporter: Kyle M Hall <kyle.m.hall>
Component: CirculationAssignee: Kyle M Hall <kyle.m.hall>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, nengard, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 7526 - longoverdue.pl leaves items marked as lost as still checked out to patron
Bug 7526 - longoverdue.pl leaves items marked as lost as still checked out to patron
Bug 7526 - longoverdue.pl leaves items marked as lost as still checked out to patron
[SIGNED-OFF] Bug 7526 - longoverdue.pl leaves items marked as lost as still checked out to patron

Description Kyle M Hall 2012-02-10 15:11:16 UTC
When the longoverdu.pl script is run, and it marks an item as lost ( using LostItem() ), if fails to remove the item from the borrower record. So, the item is marked as lost, but is also still listed as checked out to the borrower.
Comment 1 Kyle M Hall 2012-02-10 15:16:24 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2012-02-10 15:49:06 UTC
The current functionality is the way we've always expected it to work! Maybe longoverdue.pl needs an option added to it in order to trigger the the process you want?
Comment 3 Kyle M Hall 2012-02-10 17:13:07 UTC
(In reply to comment #2)

Agreed, if this was expected functionality, I will modify my patch and make it a system preference defaulting to the original functionality. Any suggestions for a preference name? RemoveLostItemsFromPatronAccount?

> The current functionality is the way we've always expected it to work! Maybe
> longoverdue.pl needs an option added to it in order to trigger the the process
> you want?
Comment 4 Owen Leonard 2012-02-10 17:22:37 UTC
longoverdue.pl is always called from the command line isn't it? So wouldn't a command line parameter be sufficient?
Comment 5 Kyle M Hall 2012-02-10 17:35:09 UTC
(In reply to comment #4)

Good point, I'll make it a command line switch instead.

> longoverdue.pl is always called from the command line isn't it? So wouldn't a
> command line parameter be sufficient?
Comment 6 Kyle M Hall 2012-02-10 17:44:47 UTC Comment hidden (obsolete)
Comment 7 Kyle M Hall 2012-02-10 17:48:31 UTC Comment hidden (obsolete)
Comment 8 Nicole C. Engard 2012-03-16 20:05:05 UTC
Created attachment 8244 [details] [review]
[SIGNED-OFF] Bug 7526 - longoverdue.pl leaves items marked as lost as still checked out to patron

When the longoverdu.pl script is run, and it marks an item as lost ( using
LostItem() ), if fails to remove the item from the borrower record. So, the
item is marked as lost, but is also still listed as checked out to the
borrower.

This commit adds the command line parameter --mark-returned. If used,
longoverdue.pl will remove lost items from the borrowers record.
Functionality will remain the same if it is not used.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>

http://bugs.koha-community.org/show_bug.cgi?id=7426
Comment 9 Ian Walls 2012-03-19 12:21:31 UTC
Cleanly adds new commandline param to longoverdue.pl to make use of the new functionality.  Won't regress any behaviour for libraries who are used to the current way of things.  Cleans up some spacing in the GetOpt area.  Marking as Passed QA.
Comment 10 Jared Camins-Esakov 2012-12-31 00:43:02 UTC
There have been no further reports of problems so I am marking this bug resolved.