]> git.koha-community.org Git - koha.git/commit
Bug 25508: Only return renewal outcomes to the controller
authorMartin Renvoize <martin.renvoize@ptfs-europe.com>
Wed, 24 Mar 2021 16:38:32 +0000 (16:38 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Thu, 22 Apr 2021 13:31:28 +0000 (15:31 +0200)
commitb51efada26130d16264486514c59e8dca12639a9
tree7d38658d7211e8a5b7190643c99abf67e0ccf8d1
parent8637c8ecbd9ec5eaed6854b55c803d5689d22868
Bug 25508: Only return renewal outcomes to the controller

There are a few cases where the `renew_item` method in
Koha::Account::Line will return `undef`. For these cases, we should not
pass the error up the chain to the controllers as it leads to malformed
error messages in the UI.

Test plan
1 - Make sure FinesMode is on, RenewAccruingItemWhenPaid is off
2 - Checkout an item to a patron and make it overdue (can backdate the
    checkout)
3 - Make sure the itemtype has fines that will be charged
4 - Charge the fines:
    Set finesMode = production
    perl misc/cronjobs/fines.pl -v
5 - Check the fine appears on the patrons account
6 - Pay off the fine
7 - Receive alert after payment that reads: "The fines on
    the following items were paid off, renewal results are
    displayed below:
        No title ( ): Not renewed - Unknown error"
8 - Apply the patch
9 - Repeat steps 1 through 6 and note that you no longer trigger the
    error message.

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit d4dc15ee36dc108e074e4d1b78edc055b1a22597)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Koha/Account.pm
Koha/Account/Line.pm