Bug 12943

Summary: Add patron branchcode to patron search results for holds
Product: Koha Reporter: Kyle M Hall (khall) <kyle>
Component: Hold requestsAssignee: Kyle M Hall (khall) <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: gmcharlt, jonathan.druart, tomascohen
Version: unspecified   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 12943 - Add patron branchcode to patron search results for holds
[SIGNED OFF] Bug 12943 - Add patron branchcode to patron search results for holds
Bug 12943 - Add patron branchcode to patron search results for holds

Description Kyle M Hall (khall) 2014-09-17 14:46:50 UTC
For some libraries, the current information specified in the patron results when search for a patron to place hold is insufficient. These libraries would like the branchcode to be added to the results to help identify the correct patron.
Comment 1 Kyle M Hall (khall) 2014-09-17 14:49:23 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall (khall) 2014-09-17 14:51:03 UTC
It should be noted that this patch brings the patron search results for holds in line with the patron search results for checkouts. In that sense, this could be considered a bug fix.
Comment 3 Nick Clemens (kidclamp) 2014-09-20 18:36:27 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2014-09-26 07:49:31 UTC
My first filling was to want the branchname instead of the branchcode. But this will add 1 DB request per result.
Comment 5 Jonathan Druart 2014-09-26 07:49:56 UTC
Created attachment 31888 [details] [review]
Bug 12943 - Add patron branchcode to patron search results for holds

For some libraries, the current information specified in the patron
results when search for a patron to place hold is insufficient. These
libraries would like the branchcode to be added to the results to help
identify the correct patron.

Test Plan:
1) Apply this patch
2) Browse to the holds for a record
3) Search for patrons from the record holds screen
4) Note the patron results now have the branchcode after the patron type

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 6 Tomás Cohen Arazi (tcohen) 2014-11-21 19:02:49 UTC
Patch pushed to master.

Thanks Kyle!