[Koha-bugs] [Bug 15895] Add Koha::Account module, use Koha::Account::pay internally for recordpayment

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

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

--- Comment #7 from Kyle M Hall <kyle at bywatersolutions.com> ---
(In reply to Jonathan Druart from comment #6)
> (In reply to Kyle M Hall from comment #4)
> > As an aside, maybe we should just add a reset() ( or maybe refresh() )
> > method to Koha::Object so we don't have to go through _result() which is
> > specific to our implementation and breaks encapsulation.
> 
> Just a thought: should not we update Koha::Object->store to make it returns
> an updated/refreshed object?

It's not a bad idea, but given that we've not needed such behavior yet, it
would probably be best to make it an optional method. I imagine there would be
an associated cost for refetching the data from the database. I would be good
to run some benchmarks, if there is no appreciable speed difference I'd say we
should definitely do what you propose!

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