[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 #15 from Kyle M Hall <kyle at bywatersolutions.com> ---
(In reply to Srdjan Jankovic from comment #14)
> 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.

I understand. Due to the complexity of the bug series (
https://bugs.koha-community.org/bugzilla3/showdependencygraph.cgi?id=15895 )
I'd like to wait until the patch set has all been pushed to master or at least
passed qa. At that time I can submit a bug and patch to make this change.

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