Bug 13549 - Add cardnumber to overdue circulation report overdue.pl
Summary: Add cardnumber to overdue circulation report overdue.pl
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-01-09 21:18 UTC by Marjorie Barry-Vila
Modified: 2016-01-19 04:36 UTC (History)
6 users (show)

See Also:
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 (99.11 KB, image/png)
2015-01-09 21:18 UTC, Marjorie Barry-Vila 		Details
Bug 13549: Add cardnumber to overdue circulation report overdue.pl (2.80 KB, patch)
2015-01-13 03:33 UTC, Justin Vos 		Details | Diff | Splinter Review
[SIGNED OFF] Bug 13549: Add cardnumber to overdue circulation report overdue.pl (2.86 KB, patch)
2015-01-13 07:12 UTC, Katrin Fischer 		Details | Diff | Splinter Review
[PASSED QA] Bug 13549: Add cardnumber to overdue circulation report overdue.pl (2.92 KB, patch)
2015-01-15 09:03 UTC, Martin Renvoize (ashimema) 		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 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!