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

--- Comment #72 from Marcel de Rooy <[email protected]> ---
(In reply to Jacek Ablewicz from comment #62)
> Note: for ensuring that MaxFine behaviour is unchanged, I did just:
> 
> 1) set MaxFine to 100.00
> 2) commented out $accountline->set(...) in UpdateFine
> 3) (without the last patch): fines.pl 2>warnings.01 - I got over 1200
> "Reducing fine for ... MaxFine reached" warnings in warnings.01 for my test
> database with 19k overdues
> 4) (with the last patch included): fines.pl 2>warnings.02
> 5) diff warnings.01 warnings.02: no differences
> 
> Some extra testing with step 2) excluded would be better, but hopefully the
> above is good enough. Also in my opinion code changes introduced by the last
> patch are 100% solid and regression-free.

Thx for your signoff and additional testing.

-- 
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/

Reply via email to