Bug 31120: Items will renew for zero ( 0 ) days if renewalperiod is blank/empty value
authorKyle M Hall <kyle@bywatersolutions.com>
Thu, 7 Jul 2022 19:14:59 +0000 (15:14 -0400)
committerTomas Cohen Arazi <tomascohen@theke.io>
Wed, 27 Jul 2022 13:03:37 +0000 (10:03 -0300)
commit9b010d99dfcb1a77a6ebd679aabafd819c6e67cf
tree2bc75a99e18f6e8a89d9f247ef654496cf48e67b
parent511ae8298e033ddf2a12799c968d25bd716d62af
Bug 31120: Items will renew for zero ( 0 ) days if renewalperiod is blank/empty value

If the rule renewalperiod is the blank empty string instead of being null/undefined or non-existant, Koha will interpret the renewal period as being zero days instead of falling back to the issuelength rule.

It makes sense to me that a literal 0 here should make it renew for zero days even though that is nonsensical.

Test Plan:
1) Delete all your rules
2) Create an all/all/all rules with an empty string for renewal base
   period
3) Note that renewing an item does nothing
4) Apply this patch
5) Restart all the things!
6) Renew again
7) Note the renewal uses the issuelength rule as intended

Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
C4/Circulation.pm