Bug 16145

Summary: Regression: Bug 15632 broke display of library name on circulation messages
Product: Koha Reporter: Owen Leonard <oleonard>
Component: CirculationAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: brendan, gmcharlt, jonathan.druart, julian.maurice, kyle.m.hall, sandboxes
Version: Main   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/circ/circulation.pl?borrowernumber=X
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 15632    
Bug Blocks:    
Attachments: Bug 16145: Regression: Bug 15632 broke display of library name on circulation messages
Bug 16145: Regression: Bug 15632 broke display of library name on circulation messages
[PASSED QA] Bug 16145: Regression: Bug 15632 broke display of library name on circulation messages

Description Owen Leonard 2016-03-24 12:07:36 UTC
Before Bug 15632 if you added a message to a patron's account in circulation the message would display with a library name. Now the library name doesn't appear.
Comment 1 Jonathan Druart 2016-03-24 21:29:28 UTC Comment hidden (obsolete)
Comment 2 Biblibre Sandboxes 2016-03-24 22:17:17 UTC
Patch tested with a sandbox, by Christopher Brannon <cbrannon@cdalibrary.org>
Comment 3 Biblibre Sandboxes 2016-03-24 22:17:38 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2016-03-24 22:31:29 UTC
Created attachment 49573 [details] [review]
[PASSED QA] Bug 16145: Regression: Bug 15632 broke display of library name on circulation messages

Wrong var loop used!

Test plan:
Add a message for a patron on the checkouts page.
The library name should be displayed.

Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Brendan Gallagher 2016-03-25 21:11:54 UTC
Pushed to Master - Should be in the May 2016 Release.  Thanks!
Comment 6 Julian Maurice 2016-04-07 07:58:02 UTC
Not needed in stable versions, status changed to RESOLVED