[Koha-bugs] [Bug 14597] Reverting a batch where a record overlaid is now deleted record will fail

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14597

--- Comment #12 from Srdjan Jankovic <srdjan at catalyst.net.nz> ---
Since I was kindly ask to take part in this discussion:

* Koha::Log lacks a bit in the Synopsis department, especially examples how to
use it within Koha:: (object) and C4 (procedural) namespaces

* I don't mind the way it is used. If the decision was made to go with 
Log::Log4perl, then this is along its lines. An alternative would be to create
a log object in context, and use it. Both have pros and cons. I do like the
idea of having log category per request (one logger in context) as opposed to
per function (which is the case with this patch). A more complete solution
would be to add a mechanism to add to category, so in this case whole request
would be eg marc-import, and this particular category would be
marc-import.C4.ImportBatch.BatchRevertRecords

-- 
You are receiving this mail because:
You are watching all bug changes.