Bug 22090

Summary: Cash register report missing data in CSV export
Product: Koha Reporter: Andrew Fuerste-Henry <andrewfh>
Component: ReportsAssignee: Katrin Fischer <katrin.fischer>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: katrin.fischer, lucas, martin.renvoize, nick
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:
19.05.00, 18.11.05
Attachments: Bug 22090: Add missing information to cash register CSV file
Bug 22090: Add missing information to cash register CSV file
Bug 22090: Add missing information to cash register CSV file

Description Andrew Fuerste-Henry 2019-01-09 20:46:01 UTC
When exporting a cash_register_stats.pl report, the following fields are missing: 
manager surname
patron surname
note
Comment 1 Katrin Fischer 2019-04-08 00:24:31 UTC
Created attachment 87487 [details] [review]
Bug 22090: Add missing information to cash register CSV file

Some information was missing from the CSV export file
of the Cash register report: manager surname, patron surname
and payment notes.

Also pretties up the column headers a bit and makes the
sequence between GUI and CSV export match.

To test:
- Make sure you have some fines and fees and payments in
  your system
- Make a search with the Cash register report
- Export results as CSV
- Proof read hader line and make sure contents match headings
- Verify first name and surname are shown for manager and patron
- Verify notes show up
Comment 2 Michal Denar 2019-04-08 19:11:50 UTC
Created attachment 87548 [details] [review]
Bug 22090: Add missing information to cash register CSV file

Some information was missing from the CSV export file
of the Cash register report: manager surname, patron surname
and payment notes.

Also pretties up the column headers a bit and makes the
sequence between GUI and CSV export match.

To test:
- Make sure you have some fines and fees and payments in
  your system
- Make a search with the Cash register report
- Export results as CSV
- Proof read hader line and make sure contents match headings
- Verify first name and surname are shown for manager and patron
- Verify notes show up

Signed-off-by: Michal Denar <black23@gmail.com>
Comment 3 Chris Cormack 2019-04-14 22:28:59 UTC
Created attachment 87965 [details] [review]
Bug 22090: Add missing information to cash register CSV file

Some information was missing from the CSV export file
of the Cash register report: manager surname, patron surname
and payment notes.

Also pretties up the column headers a bit and makes the
sequence between GUI and CSV export match.

To test:
- Make sure you have some fines and fees and payments in
  your system
- Make a search with the Cash register report
- Export results as CSV
- Proof read hader line and make sure contents match headings
- Verify first name and surname are shown for manager and patron
- Verify notes show up

Signed-off-by: Michal Denar <black23@gmail.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Comment 4 Nick Clemens (kidclamp) 2019-04-16 13:25:53 UTC
Awesome work all!

Pushed to master for 19.05
Comment 5 Martin Renvoize 2019-04-25 14:03:31 UTC
Pushed to 18.11.x for 18.11.05
Comment 6 Lucas Gass 2019-05-10 20:08:05 UTC
backported to 18.05.x for 18.05.13