Bug 6403

Summary: Record local use stats when checking in
Product: Koha Reporter: Ian Walls <koha.sekjal>
Component: CirculationAssignee: Ian Walls <koha.sekjal>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: enhancement    
Priority: PATCH-Sent (DO NOT USE) CC: chris, gmcharlt, nengard, paul.poulain
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: Sponsored Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Proposed patch (functionality)
Proposed Patch (DB changes)
Proposed Patch (DB changes)
[SIGNED-OFF] Enh 6403: Record local use stats when checking in unissued materials
[SIGNED-OFF] Enh 6403 DB changes: add RecordLocalUseOnReturn syspref

Description Ian Walls 2011-05-24 16:42:01 UTC
A new system preference, RecordInhouseUseAtCheckin, which will add a 'local-use' statistical entry for items when they are checked in.  This will only apply when:

1.  the system preference is on (obviously)
2.  the item is not checked out
3.  the item is not on hold or up for transfer

That is, this statistical use will only occur in the situation where the "not checked out" error message would be displayed.  This message shall be added onto to indicate that statistical usage was recorded.

The borrowernumber for the staff member doing the checkin shall be entered into the statistics table (rather than the borrowernumber for a statistical usage patron)
Comment 1 Ian Walls 2011-09-22 20:30:47 UTC Comment hidden (obsolete)
Comment 2 Ian Walls 2011-09-22 20:31:23 UTC Comment hidden (obsolete)
Comment 3 Nicole C. Engard 2011-09-26 14:18:49 UTC Comment hidden (obsolete)
Comment 4 Nicole C. Engard 2011-09-26 14:26:40 UTC
Created attachment 5588 [details] [review]
[SIGNED-OFF] Enh 6403: Record local use stats when checking in unissued materials

Adds a new system preference, RecordLocalUseOnReturn, which when active will change the statistical
entry type from "return" to "localuse" in AddReturn() if the material was not on loan when returned.
The intended use-case here is for libraries with 'open' book drops, in which patrons can put locally
used (but unissued) materials.

Adds a small message to the user interface to confirm that Local Use was recorded.

This change opens up the possibility to record more types of statistics on return; one would just need
to update the $stat_type variable accordingly.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Comment 5 Nicole C. Engard 2011-09-26 14:28:21 UTC
Created attachment 5589 [details] [review]
[SIGNED-OFF] Enh 6403 DB changes: add RecordLocalUseOnReturn syspref

This patch, separate from the main functionality patch, adds the new syspref to the sysprefs.sql
as well as updatedatabase.pl

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Comment 6 Paul Poulain 2011-10-07 14:52:37 UTC
QA comment
  * good test plan
  * works as expected
  * nothing to say about the code itself

My only comment could be that some libraries consider saving such returns as a bug, so we could set the default value of the new syspref to 1 directly.
But we've a rule to say a new syspref sould not change the behaviour for libraries that don't do anything with it.

SO :
* passed QA
* it's worth being very clear in the release note that most libraries will probably want to set this value to 1 after upgrading
Comment 7 Chris Cormack 2011-10-09 06:54:57 UTC
Pushed, please test
Comment 8 Jared Camins-Esakov 2012-05-23 12:10:31 UTC
This appears to be working. Closing.