[Koha-bugs] [Bug 17586] Add the Koha::Account::Lines->get_balance method

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Dec 6 13:04:12 CET 2016


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

--- Comment #15 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
(In reply to Kyle M Hall from comment #14)
> (In reply to Jonathan Druart from comment #13)
> > (In reply to Kyle M Hall from comment #10)
> > > I would do $patron->account()->balance() with account() returning the
> > > Koha::Account object, and balance() returning the account balance. Of
> > > course, I'm a fan of being succinct ; )
> > 
> > I am not 100% this change is the right thing to do but we could go like that
> > to start.
> > For now, the Koha::Account module's responsibilities are not clearly defined
> > and it's not easy to guess what it will become.
> 
> That's fine. I just wanted to put that thought out. We may have a better
> notion of where such a method would go after the fact and we can make that
> change at the end rather then have to modify all the dependent patches.

I have modified the dependent patches, so let's go with it :)

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


More information about the Koha-bugs mailing list