https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15895
--- Comment #15 from Kyle M Hall <[email protected]> --- (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. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
