Attachment 143984 Details for Bug 30794 – Bug 30794: Insert blank rule (unlimited) for max_holds

[patch] Bug 30794: Insert blank rule (unlimited) for max_holds

Bug-30794-Insert-blank-rule-unlimited-for-maxholds.patch (text/plain), 1.11 KB, created by Jonathan Druart on 2022-11-17 08:30:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2022-11-17 08:30:50 UTC

Size: 1.11 KB

patch

obsolete

>From e994bc05f87bb8d1a769ab4aadbaaed3790c1bb8 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Thu, 17 Nov 2022 09:28:39 +0100
>Subject: [PATCH] Bug 30794: Insert blank rule (unlimited) for max_holds
>
>Related to bug 24669, why didn't we use strip_non_numeric for one occurrence?
>
>Note that there is still a problem, if none of the 3 inputs is filled,
>no rule will be created (ie. action ignored)
>---
> admin/smart-rules.pl | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
>diff --git a/admin/smart-rules.pl b/admin/smart-rules.pl
>index b71179437fe..7de8899b285 100755
>--- a/admin/smart-rules.pl
>+++ b/admin/smart-rules.pl
>@@ -415,8 +415,7 @@ elsif ($op eq "add-branch-cat") {
>     my $patron_maxonsiteissueqty = $input->param('patron_maxonsiteissueqty');
>     $patron_maxonsiteissueqty = strip_non_numeric($patron_maxonsiteissueqty);
>     my $max_holds = $input->param('max_holds');
>-    $max_holds =~ s/\s//g;
>-    $max_holds = undef if $max_holds !~ /^\d+/;
>+    $max_holds = strip_non_numeric($max_holds);
> 
>     if ($branch eq "*") {
>         if ($categorycode eq "*") {
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 30794: 143984 | 144011 | 144050