From e749afc938615f273d2966aed31fe387d8d36b64 Mon Sep 17 00:00:00 2001
From: Josef Moravec <josef.moravec@gmail.com>
Date: Wed, 24 Apr 2019 21:11:29 +0000
Subject: [PATCH] Bug 18928: (follow-up) Unset max_holds rule if needed

Test plan:
1) Set all values in Default checkout, hold and return policy section on
circulation rules admin page, try for all libraries and for one specific
library. Save.
2) Click on unset and confitm
--> without patch, the Maximum total holds allowed value remains
--> with patch the unset removex Maximum holds value too

Signed-off-by: Agustin Moyano <agustinmoyano@theke.io>
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
---
 admin/smart-rules.pl | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/admin/smart-rules.pl b/admin/smart-rules.pl
index 41c61f733d..2bdebc9186 100755
--- a/admin/smart-rules.pl
+++ b/admin/smart-rules.pl
@@ -104,6 +104,7 @@ elsif ($op eq 'delete-branch-cat') {
                          holdallowed             => undef,
                          hold_fulfillment_policy => undef,
                          returnbranch            => undef,
+                         max_holds               => undef,
                      }
                  }
              );
@@ -133,6 +134,7 @@ elsif ($op eq 'delete-branch-cat') {
                     holdallowed             => undef,
                     hold_fulfillment_policy => undef,
                     returnbranch            => undef,
+                    max_holds               => undef,
                 }
             }
         );
-- 
2.39.5