Bug 17925

Summary: Disable debugging in reports/bor_issues_top.pl
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: ReportsAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: katrin.fischer, kyle, mtj, mtompset, veron
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 17925: Disable debugging in reports/bor_issues_top.pl
Bug 17925: Disable debugging in reports/bor_issues_top.pl
Bug 17925: Disable debugging in reports/bor_issues_top.pl
Bug 17925: Disable debugging in reports/bor_issues_top.pl

Description Marcel de Rooy 2017-01-17 15:49:23 UTC

    
Comment 1 Marcel de Rooy 2017-01-17 15:57:35 UTC Comment hidden (obsolete)
Comment 2 Mark Tompsett 2017-01-19 01:10:40 UTC
Eyeballs well.
Comment 3 Marcel de Rooy 2017-01-19 08:13:34 UTC
Created attachment 59208 [details] [review]
Bug 17925: Disable debugging in reports/bor_issues_top.pl

Remove setting $debug to 1. This prevents creation of file
tmp/bor_issues_top.debug.log.
Enable warnings pragma.
Resolve a uninitialized warning on $sep like:
Use of uninitialized value $CGI::Compile::ROOT::usr_share_koha_masterclone_reports_bor_issues_top_2epl::sep in string eq at /usr/share/koha/masterclone/reports/bor_issues_top.pl line 66.

Test plan:
Restart Plack (as you always do when testing).
Remove file /tmp/bor_issues_top.debug.log.
Open Reports/Patrons with the most checkouts.
Check if the file in /tmp has not been created again.
Check the log for warnings after you restarted Plack.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Marc VĂ©ron 2017-05-02 03:48:32 UTC
Something weird while testing, not related to this bug but I write it down for the records:

The column 'Patron' contains values like
surname,[tab]first name
e.g.
Meier,	Hans

That will break any csv import using tab or , as separator
Comment 5 Mason James 2017-05-10 04:46:38 UTC
Created attachment 63316 [details] [review]
Bug 17925: Disable debugging in reports/bor_issues_top.pl

Remove setting $debug to 1. This prevents creation of file
tmp/bor_issues_top.debug.log.
Enable warnings pragma.
Resolve a uninitialized warning on $sep like:
Use of uninitialized value $CGI::Compile::ROOT::usr_share_koha_masterclone_reports_bor_issues_top_2epl::sep in string eq at /usr/share/koha/masterclone/reports/bor_issues_top.pl line 66.

Test plan:
Restart Plack (as you always do when testing).
Remove file /tmp/bor_issues_top.debug.log.
Open Reports/Patrons with the most checkouts.
Check if the file in /tmp has not been created again.
Check the log for warnings after you restarted Plack.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Mason James <mtj@kohaaloha.com>
Comment 6 Mason James 2017-05-10 04:47:28 UTC
seems to work as expected, thanks Marcel!
Comment 7 Jonathan Druart 2017-05-10 12:14:17 UTC
Created attachment 63337 [details] [review]
Bug 17925: Disable debugging in reports/bor_issues_top.pl

Remove setting $debug to 1. This prevents creation of file
tmp/bor_issues_top.debug.log.
Enable warnings pragma.
Resolve a uninitialized warning on $sep like:
Use of uninitialized value $CGI::Compile::ROOT::usr_share_koha_masterclone_reports_bor_issues_top_2epl::sep in string eq at /usr/share/koha/masterclone/reports/bor_issues_top.pl line 66.

Test plan:
Restart Plack (as you always do when testing).
Remove file /tmp/bor_issues_top.debug.log.
Open Reports/Patrons with the most checkouts.
Check if the file in /tmp has not been created again.
Check the log for warnings after you restarted Plack.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Mason James <mtj@kohaaloha.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 8 Kyle M Hall 2017-05-12 13:09:27 UTC
Pushed to master for 17.05, thanks Marcel!
Comment 9 Katrin Fischer 2017-05-14 10:17:03 UTC
This patch has been pushed to 16.11.x and will be in 16.11.08.
Comment 10 Mason James 2017-05-24 02:39:57 UTC
Pushed to 16.05.x, for 16.05.13 release