[Koha-bugs] [Bug 14167] Add Koha::Logger based on Log4perl
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Fri Jun 26 15:59:49 CEST 2015
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14167
--- Comment #41 from Kyle M Hall <kyle at bywatersolutions.com> ---
Created attachment 40662
-->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=40662&action=edit
[SIGNED-OFF] Bug 14167: [QA Follow-up] Making Koha::Logger bit more crash
resistant
Moving the BEGIN block to _init for the most part. We only need to
initialize when we actually start using the logger.
Removed the third init part. If we do not have a log4perl_conf in the
koha config, we are not using it yet.
Method get uses hash parameters now. It calls init. If we do not have a
config or the logfile is not writable, we will not use log4perl.
Using AUTOLOAD as a wrapper around Log4perl in order to add some checks
that log4perl does not have.
If a logrotate would change file permissions on a default logfile, we
should catch that now too (see recheck).
Test plan:
Run the previous tests again.
Will still add a unit test.
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list