From 572847b439028f394dfd35ad00c95025890acd88 Mon Sep 17 00:00:00 2001 From: Emmi Takkinen Date: Tue, 15 Nov 2022 09:31:13 +0200 Subject: [PATCH] Bug 30694: Set decreaseloanholds undef when deleting circulation rule When one tries to delete rule with field "Decreased loan period for high holds" filled, delete fails without errors. This happens because rule isn't set in code section handling the delete in smart-rules.pl. To test: 1. Create a new circulation rule and fill all fields. 2. Try to delete the rule. => Rule isn't deleted, all fields except "Decreased loan period for high holds" are now empty or have value "Unlimited". 3. Modify the rule and set field "Decreased loan period for high holds" empty. 4. Delete the rule. => Rule is deleted successfully. 5. Create a new rule, fill all fields expect field "Decreased loan period for high holds". 6. Delete the rule. => Rule is deleted successfully. 7. Apply this patch. 8. Again create a new rule with field "Decreased loan period for high holds" filled. 9. Try to delete the rule. => Rule is deleted successfully. Sponsored-by: Koha-Suomi Oy Signed-off-by: Katrin Fischer Signed-off-by: Kyle M Hall --- admin/smart-rules.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/admin/smart-rules.pl b/admin/smart-rules.pl index 7de8899b28..b74a0ceeab 100755 --- a/admin/smart-rules.pl +++ b/admin/smart-rules.pl @@ -114,6 +114,7 @@ if ($op eq 'delete') { recall_due_date_interval => undef, recall_overdue_fine => undef, recall_shelf_time => undef, + decreaseloanholds => undef, } } ); -- 2.30.2