Bug 39153

Summary: About does not handle log4perl warnings correctly.
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: AboutAssignee: Marcel de Rooy <m.de.rooy>
Status: Pushed to main --- QA Contact: Martin Renvoize (ashimema) <martin.renvoize>
Severity: normal    
Priority: P5 - low CC: martin.renvoize
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39155
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
25.05.00
Circulation function:
Bug Depends on:    
Bug Blocks: 39154    
Attachments: Bug 39153: Move log4perl checks in about.pl
Bug 39153: Move log4perl checks in about.pl
Bug 39153: Move log4perl checks in about.pl
Bug 39153: Move log4perl checks in about.pl

Description Marcel de Rooy 2025-02-18 13:21:21 UTC
There is a check in tab about which should be moved to sysinfo..
Comment 1 Marcel de Rooy 2025-02-18 14:11:22 UTC Comment hidden (obsolete)
Comment 2 Marcel de Rooy 2025-02-18 14:15:12 UTC
Created attachment 178269 [details] [review]
Bug 39153: Move log4perl checks in about.pl

They were mostly under tab about, but they should be under sysinfo.

Test plan:
If you would move your log4perl file, plack.psgi crashes. So sorry
but we cannot test that.
Note that you cannot simulate logfile_not_writable either, since it
also crashes plack.psgi before reaching about.pl. See 39155.

So a trivial hack remains:
Bonus: Edit about.pl and changed the moved if( !-w $file ) test and
replace it by if ( 1 ) in the new sub log4perl_check.
Restart and verify that you now have an error message on sysinfo.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 3 David Nind 2025-02-18 17:53:46 UTC
Created attachment 178299 [details] [review]
Bug 39153: Move log4perl checks in about.pl

They were mostly under tab about, but they should be under sysinfo.

Test plan:
If you would move your log4perl file, plack.psgi crashes. So sorry
but we cannot test that.
Note that you cannot simulate logfile_not_writable either, since it
also crashes plack.psgi before reaching about.pl. See 39155.

So a trivial hack remains:
Bonus: Edit about.pl and changed the moved if( !-w $file ) test and
replace it by if ( 1 ) in the new sub log4perl_check.
Restart and verify that you now have an error message on sysinfo.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: David Nind <david@davidnind.com>
Comment 4 Martin Renvoize (ashimema) 2025-03-07 12:22:22 UTC
Created attachment 179071 [details] [review]
Bug 39153: Move log4perl checks in about.pl

They were mostly under tab about, but they should be under sysinfo.

Test plan:
If you would move your log4perl file, plack.psgi crashes. So sorry
but we cannot test that.
Note that you cannot simulate logfile_not_writable either, since it
also crashes plack.psgi before reaching about.pl. See 39155.

So a trivial hack remains:
Bonus: Edit about.pl and changed the moved if( !-w $file ) test and
replace it by if ( 1 ) in the new sub log4perl_check.
Restart and verify that you now have an error message on sysinfo.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 5 Katrin Fischer 2025-03-07 14:42:57 UTC
Pushed for 25.05!

Well done everyone, thank you!