Bug 28570

Summary: bor_issues_top.pl using a /tmp file to log debug
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: fridolin.somers, kyle, martin.renvoize, victor
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.11.00,21.05.02,20.11.09
Bug Depends on:    
Bug Blocks: 28572    
Attachments: Bug 28570: Remove log to /tmp file from bor_issues_top.pl
Bug 28570: Remove log to /tmp file from bor_issues_top.pl
Bug 28570: Remove log to /tmp file from bor_issues_top.pl

Description Jonathan Druart 2021-06-16 13:24:02 UTC
$debug and open my $debugfh, '>', '/tmp/bor_issues_top.debug.log';

This must be removed.
Comment 1 Jonathan Druart 2021-06-16 13:25:49 UTC
Created attachment 122038 [details] [review]
Bug 28570: Remove log to /tmp file from bor_issues_top.pl

This is unusual and must not be done.
Removing it.

Test plan:
Use the "Home > Reports > Patrons with the most checkouts" report and
confirm that it is working correctly after you applied this patch
Comment 2 Martin Renvoize 2021-06-18 14:27:43 UTC
Created attachment 122145 [details] [review]
Bug 28570: Remove log to /tmp file from bor_issues_top.pl

This is unusual and must not be done.
Removing it.

Test plan:
Use the "Home > Reports > Patrons with the most checkouts" report and
confirm that it is working correctly after you applied this patch

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 3 Martin Renvoize 2021-06-18 14:28:52 UTC
Clear improvement, no regressions found.. Signing off.
Comment 4 Nick Clemens (kidclamp) 2021-06-18 15:32:16 UTC
Created attachment 122163 [details] [review]
Bug 28570: Remove log to /tmp file from bor_issues_top.pl

This is unusual and must not be done.
Removing it.

Test plan:
Use the "Home > Reports > Patrons with the most checkouts" report and
confirm that it is working correctly after you applied this patch

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 5 Jonathan Druart 2021-06-22 10:14:48 UTC
Pushed to master for 21.11, thanks to everybody involved!
Comment 6 Kyle M Hall 2021-07-09 12:21:20 UTC
Pushed to 21.05.x for 21.05.02
Comment 7 Fridolin Somers 2021-08-02 20:12:43 UTC
Pushed to 20.11.x for 20.11.09
Comment 8 Victor Grousset/tuxayo 2021-08-12 21:36:27 UTC
Not backported to oldoldstable (20.05.x). Feel free to ask if it's needed.