[Koha-bugs] [Bug 21720] Update C4::Circulation::AddIssuingCharge to use Koha::Account->add_debit

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

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

Josef Moravec <josef.moravec at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |josef.moravec at gmail.com

--- Comment #7 from Josef Moravec <josef.moravec at gmail.com> ---
(In reply to Martin Renvoize from comment #6)
> QA Question: Regarding the side effect, this is reasonably easy to avert
> within the add_debit method but I'd like input as to whether the FinesLog
> should really be just about Fines or should include all Charges.. as it
> stands it's a bit of a mix of both.. this patch series turns it into more of
> the ChargesLog but upping the logging but it could alternatively go the
> other way and make what is recorded a bit more explicitly about fines and
> only fines.

I would prefer to log all charges, not only fines.

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