]> git.koha-community.org Git - koha.git/commit
Bug 25758: Return on_reserve over too_soon when not calling from automatic_renewals...
authorNick Clemens <nick@bywatersolutions.com>
Fri, 4 Sep 2020 12:47:58 +0000 (12:47 +0000)
committerAleisha Amohia <aleishaamohia@hotmail.com>
Tue, 24 Nov 2020 00:01:39 +0000 (13:01 +1300)
commit321afcfe745e1423f106bfa435e452191c4a7fd7
treee97ed192538f11395c72f30824c51dc57c4670e4
parent4a2fcd6604ec4f7490334ba98b038d46bf3a8cfb
Bug 25758: Return on_reserve over too_soon when not calling from automatic_renewals cron

Bug 19014 altered CanBookBeRenewed to return (auto_)too_soon over on_reserve

For cron purposes this is the correct behaviour.

For display purposes we wish to see on_reserve over too_soon

This patchset adds a switch to 'CanBookBeRenewed' to alter the priority of these statuses

To test:
 1 - set NoRenewalBeforePrecision to date only
 2 - set a circ rule to auto-renewal=yes, no renewal before=0, checkout period to 7 days
 3 - check item out
 4 - confirm item shows Scheduled For Automatic Renewal in issues table
 5 - place a hold on the item for another patron
 6 - reload issues table for patron 1, confirm checkout still shows "scheduled for automatic renewal" rather than "on hold"
 7 - change No Renewal Before value to 7
 8 - reload issues table for patron 1, confirm checkout now shows "on hold"
 9 - Apply patch
10 - restart_all
11 - Reload the issues table - confirm 'on_hold' still shows
12 - Change No Renewal Before to 0
13 - Refresh issues table, still shows 'On hold'
14 - perl misc/cronjobs/automatic_renewals.pl -v
15 - Result shows 'auto_too_soon'
16 - prove -v t/db_dependent/Circulation.t

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>
C4/Circulation.pm
misc/cronjobs/automatic_renewals.pl
t/db_dependent/Circulation.t