https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35907

--- Comment #15 from Marcel de Rooy <[email protected]> ---
(In reply to Kyle M Hall from comment #0)
> Because of the way Koha::Logger has been used to log to different categories
> based on the interface and caller, it can be extremely hard to log all of a
> particular log statement to one place.

Just change your log4perl.conf configuration? And a sysadmin could always
concatenate all logs and grep on whatever he wants to find?

This works as expected but feels a bit like a workaround.

+    my $prefix = $params->{prefix} // 1;
I would rather rename this one personally. What you want here is to suppress
the plack prefix, not add a custom prefix.
no_plack => 1 or something like that ?

If the log4perl config for reports.execute.query is not found, it seems that
Logger just silently ignores logging. This costed me about 15ms per call.
Probably 
 acceptable.

Passing QA since it is just a few lines. But hesitantly since our code doesnt
get cleaner.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to