https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19871 Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #61 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- (In reply to Jonathan Druart from comment #60)
Yes I think we need a "don't look at this, Joubu" keyword.
LOL. Your input is always valuable. There's a clear recurring timing issue. Can we move the discussion into bug 42391? I already refactored the core of this patchset there. I already cleaned the POD there. It is only missing a new placement for the exception mapper (?) class. I personally don't like the `Koha::Exceptions::*` namespace as it suggests it is an exception class (which is not). And I'd like to keep the helper class within the Koha::Schema context, as its domain is really DBIC. I didn't like the `Util` part, though. Maybe `Koha::Schema::ExceptionMapper` would be good enough. But as I said, can we move this conversation to the follow-up bug, which is the cherry on top for this enhancement. -- You are receiving this mail because: You are watching all bug changes.