Bug 30167: Use returned renewal date rather than fetching
authorNick Clemens <nick@bywatersolutions.com>
Wed, 23 Feb 2022 16:09:07 +0000 (16:09 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Thu, 5 May 2022 00:29:23 +0000 (14:29 -1000)
commit175182f064ffb51ee3504a528db1f7f06a443a77
tree0dd3c243f131370a1906ef2cca635c316e4d91ae
parent4cf759259ffbf5bf74af43e06b3be780c90161b2
Bug 30167: Use returned renewal date rather than fetching

This patch updates the three scripts that fetched the soonest renewal date
to use the return from CanBookBeRenewed

To test:
1 - Set a circulation rule with a 'no renewal before' set to 3, loan length set to 5
2 - Check out an item to a patron that uses this rule
3 - Verify the checkouts for the patron show the correct 'No renewal before' date
4 - Sign in to the patron's opac account
5 - Verify the item shows it cannot be renewed, and shows the correct date
6 - Go to Circulation->Renew
7 - Attempt to renew using barcode
8 - Confirm error shows the soonest renewal date

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
circ/renew.pl
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
opac/opac-user.pl
svc/checkouts