Bug 18919

Summary: "Transaction Branch" select field broken in Cash register statistics
Product: Koha Reporter: Baptiste <baptiste.wojtkowski>
Component: ReportsAssignee: Baptiste <baptiste.wojtkowski>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: fridolin.somers, katrin.fischer, lisettepalouse+koha
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 18919: Repair "Transaction branch" in cash_register_stats.pl
Bug 18919: Repair "Transaction branch" in cash_register_stats.pl
Bug 18919: Repair "Transaction branch" in cash_register_stats.pl
Bug 18919: Repair "Transaction branch" in cash_register_stats.pl
Bug 18919: Repair "Transaction branch" in cash_register_stats.pl

Description Baptiste 2017-07-10 08:11:02 UTC
To reproduce:
domain/cgi-bin/koha/reports/cash_register_stats.pl

transaction branch
all => you can see every cashing
any branch => you can see every or no cashing (I don't know exactly on what it depends)
Comment 1 Baptiste 2017-07-10 08:13:48 UTC Comment hidden (obsolete)
Comment 2 Baptiste 2017-07-10 08:16:10 UTC Comment hidden (obsolete)
Comment 3 Baptiste 2017-07-10 08:27:02 UTC
Created attachment 64950 [details] [review]
Bug 18919: Repair "Transaction branch" in cash_register_stats.pl

- values of the option fields are now correctly loaded

TEST PLAN:
go to domain/cgi-bin/koha/reports/cash_register_stats.pl
1 - Make a report for all branches => it works
2 - Make a report for any branch => you will get the same or no result
3 - apply patch
4 - Repeat 2 => it should work
Comment 4 Alex Buckley 2017-07-27 20:38:02 UTC
Created attachment 65297 [details] [review]
Bug 18919: Repair "Transaction branch" in cash_register_stats.pl

- values of the option fields are now correctly loaded

TEST PLAN:
go to domain/cgi-bin/koha/reports/cash_register_stats.pl
1 - Make a report for all branches => it works
2 - Make a report for any branch => you will get the same or no result
3 - apply patch
4 - Repeat 2 => it should work

Followed test plan worked as intended
Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
Comment 5 Nick Clemens (kidclamp) 2017-07-31 10:36:31 UTC
Created attachment 65332 [details] [review]
Bug 18919: Repair "Transaction branch" in cash_register_stats.pl

- values of the option fields are now correctly loaded

TEST PLAN:
go to domain/cgi-bin/koha/reports/cash_register_stats.pl
1 - Make a report for all branches => it works
2 - Make a report for any branch => you will get the same or no result
3 - apply patch
4 - Repeat 2 => it should work

Followed test plan worked as intended
Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 6 Jonathan Druart 2017-08-15 15:27:48 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 7 Fridolin Somers 2017-09-06 13:39:12 UTC
Pushed to 17.05.x, will be in 17.05.04
Comment 8 Nick Clemens (kidclamp) 2017-10-31 13:06:03 UTC
*** Bug 19065 has been marked as a duplicate of this bug. ***