https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27080
--- Comment #9 from Joonas Kylmälä <[email protected]> --- (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. _______________________________________________ Koha-bugs mailing list [email protected] https://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/
