[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 #14 from Srdjan Jankovic <srdjan at catalyst.net.nz> ---
I suggested Koha::Patron for the reason that you never pay for multiple
patrons' fees. Patron->accountlines() makes sure of that.

Koha::AccountLines is better than Koha::Account because there's only one method
call.

I still think Patron->pay() is superior. The only argument against is you may
not need the borrowers record at that stage, but that's not very likely, and
can be made irrelevant with loading Patron with account lines preloaded or
something.

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