[Koha-bugs] [Bug 27080] Add --statute-barred option to cleanup_database.pl to purge all fees where the item has been returned and the fee is older than the given number of days.

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed Dec 2 12:02:41 CET 2020


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

--- Comment #9 from Joonas Kylmälä <joonas.kylmala at helsinki.fi> ---
(In reply to Timothy Alexis Vass from comment #7)
> So the scope becomes rewriting the entire C4/Accounts.pm module as a new
> module?

No, just focus on the parts that matter for this bug. It would be welcome, but
not necessary, to refactor the already existing related functions. In this case
it would be only purge_zero_balance_fees() and that would need to be moved to
some module in Koha namespace. Then the new function
purge_statute_barred_fees() could be added in later patch to go to this same
module. Or you can also go the easier route, which is to add only
purge_statute_barred_fees() to a module in Koha namespace. Then in the future
we can tidy up C4/Accounts.pm a bit by moving purge_zero_balance_fees() also to
the module in Koha namespace.

Let me know if you get stuck somewhere or need clarifications on the requested
improvements.

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


More information about the Koha-bugs mailing list