Bug 13099

Summary: "Item hasn't been transferred yet" shows pickup branch rather than holding branch
Product: Koha Reporter: Kyle M Hall <kyle>
Component: Hold requestsAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: gmcharlt, paola.rossi, tomascohen
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13099 - "Item hasn't been transferred yet" shows pickup branch rather than holding branch
Bug 13099 - "Item hasn't been transferred yet" shows pickup branch rather than holding branch
[PASSED QA] Bug 13099 - "Item hasn't been transferred yet" shows pickup branch rather than holding branch

Description Kyle M Hall 2014-10-16 15:44:32 UTC
The "Item hasn't been transferred yet" message on circulation.pl and moremember.pl in the holds table shows the pickup branch rather than holding branch.
Comment 1 Kyle M Hall 2014-10-16 15:49:07 UTC Comment hidden (obsolete)
Comment 2 Paola Rossi 2014-10-21 14:18:29 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-10-25 19:25:54 UTC
Created attachment 32742 [details] [review]
[PASSED QA] Bug 13099 - "Item hasn't been transferred yet" shows pickup branch rather than holding branch

The "Item hasn't been transferred yet" message on circulation.pl and
moremember.pl in the holds table shows the pickup branch rather than
holding branch.

Test Plan:
1) Place an item on hold for a patron
2) Check the item in to set the hold to waiting
3) Place an item level hold for another patron,
   make the pickup location be a different library
4) Load the holds tab on the patron details page for the second patron
5) Note it says "Item hasn't been transferred yet from <Pickup Library>"
6) Apply this patch
7) Repeat step 4
8) Note the library listed is now the holding library

Signed-off-by: Paola Rossi <paola.rossi@cineca.it>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described and passes tests and QA script.
Comment 4 Tomás Cohen Arazi 2014-11-20 21:20:23 UTC
Patch pushed to master.

Thanks Kyle!