Bug 13549

Summary: Add cardnumber to overdue circulation report overdue.pl
Product: Koha Reporter: Marjorie Barry-Vila <marjorie.barry-vila>
Component: CirculationAssignee: Bugs List <koha-bugs>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: chris, gmcharlt, justinvos, kyle.m.hall, martin.renvoize, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: overdue_report
Bug 13549: Add cardnumber to overdue circulation report overdue.pl
[SIGNED OFF] Bug 13549: Add cardnumber to overdue circulation report overdue.pl
[PASSED QA] Bug 13549: Add cardnumber to overdue circulation report overdue.pl

Description Marjorie Barry-Vila 2015-01-09 21:18:38 UTC
Created attachment 35096 [details]
overdue_report

Hi,

It would be fine to see borrower's cardnumber on overdue circulation report in patron column.

Thanks.

Marjorie
Comment 1 Justin Vos 2015-01-13 02:28:28 UTC
I'm working on this.
Comment 2 Justin Vos 2015-01-13 03:33:21 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2015-01-13 07:12:57 UTC Comment hidden (obsolete)
Comment 4 Martin Renvoize (ashimema) 2015-01-15 09:03:56 UTC
Created attachment 35291 [details] [review]
[PASSED QA] Bug 13549: Add cardnumber to overdue circulation report overdue.pl

Test Plan:
 - Create an overdue item.
 - Navigate to overdue circulation report (Home -> Circulation -> Overdues)
 - Notice there is no card number showing in the Patron column.
 - Apply patch.
 - Refresh page.
 - Notice there is a card number showing in the Patron column.

Signed-off-by: Justin <justinvos@live.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 5 Martin Renvoize (ashimema) 2015-01-15 09:05:20 UTC
Passing QA, passes scripts, tests all still pass and is a nice fix.. Thanks Justin (and assuming you the same Justin that also fixed utf8 in git-bz.. double thanks :) )
Comment 6 Tomás Cohen Arazi (tcohen) 2015-03-31 17:20:17 UTC
Patch pushed to master.

Thanks Justin!