Bug 12523 - Add patron email in Holds awaiting pickup circulation report
Summary: Add patron email in Holds awaiting pickup circulation report
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: simith.doliveira
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 12692
  Show dependency tree / graph
 
Reported: 2014-07-03 12:24 UTC by simith.doliveira
Modified: 2015-06-04 23:33 UTC (History)
5 users (show)

See Also:
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 12523 - Add patron email in Holds awaiting pickup circulation rapport (1.37 KB, patch)
2014-07-03 12:34 UTC, simith.doliveira 		Details | Diff | Splinter Review
[SIGNED-OFF] Bug 12523 - Add patron email in Holds awaiting pickup circulation rapport (1.48 KB, patch)
2014-08-01 16:51 UTC, Owen Leonard 		Details | Diff | Splinter Review
[PASSED QA] Bug 12523 - Add patron email in Holds awaiting pickup circulation rapport (1.55 KB, patch)
2014-08-02 19:46 UTC, Katrin Fischer 		Details | Diff | Splinter Review
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description simith.doliveira 2014-07-03 12:24:27 UTC 
Our sponsor require an adjustment to allow patron email in Holds awaiting pickup circulation rapport.

Sponsor by: ville de victoriaville
Comment 1 simith.doliveira 2014-07-03 12:34:15 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2014-08-01 16:51:58 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-08-02 19:46:41 UTC 
Created attachment 30492 [details] [review]
[PASSED QA] Bug 12523 - Add patron email in Holds awaiting pickup circulation rapport

Modified:

circ/waitingreservers.pl    - added GetFirstValidEmailAddress

Testing:

I Apply the patch

0) Put an item in hold for pickup;
1) In Circulation -> Holds awaiting pickup, validate email (column Patron);

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described and will use the first valid email address.
Comment 4 Tomás Cohen Arazi (tcohen) 2014-08-07 19:59:48 UTC 
Patch pushed to master.

Thanks Simith!