[Koha-bugs] [Bug 23463] Move C4::Items CRUD subroutines to Koha::Item

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

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

--- Comment #65 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
(In reply to Tomás Cohen Arazi from comment #64)
> About the timestamp handling: shouldn't we better implement
> Koha::Object->set_and_store of similar, that instead of overwriting the
> whole object, just sets the really changed fields, and we leave the DB do
> the heavy lifting? It looks wrong (as you also say, Jonathan) to be setting
> the timestamp manually on the ->store method (be it in the Koha::Item class,
> or as a generic approach in Koha::Object)

I have added a follow-up to the branch. Still not optimal however.
I am on a "less regressions as possible" approach here. We will certainly
want/need to improve later.

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