From 601aa111cd80cfee1574d0ffab5e20b11bcc4082 Mon Sep 17 00:00:00 2001 From: Andrew Nugged Date: Tue, 19 May 2020 11:20:41 +0300 Subject: [PATCH] Bug 25440: Fix for "uninitialized $maxsuspensiondays" in smart-rules.pl This warning emitted: Use of uninitialized value $maxsuspensiondays in string eq at /admin/smart-rules.pl line 257. But that not just undef-warning, there is broken logic, these two lines are mutually contradictory and goes one-by-one: $maxsuspensiondays = undef if $maxsuspensiondays eq q||; $maxsuspensiondays = '' if $maxsuspensiondays eq q||; Fix is simple: to make it '' if it comes undef. Signed-off-by: Martin Renvoize --- admin/smart-rules.pl | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/admin/smart-rules.pl b/admin/smart-rules.pl index ea3b1df3dd..0a33493623 100755 --- a/admin/smart-rules.pl +++ b/admin/smart-rules.pl @@ -254,9 +254,7 @@ elsif ($op eq 'add') { my $itemtype = $input->param('itemtype'); # item type my $fine = $input->param('fine'); my $finedays = $input->param('finedays'); - my $maxsuspensiondays = $input->param('maxsuspensiondays'); - $maxsuspensiondays = undef if $maxsuspensiondays eq q||; - $maxsuspensiondays = '' if $maxsuspensiondays eq q||; + my $maxsuspensiondays = $input->param('maxsuspensiondays') || ''; my $suspension_chargeperiod = $input->param('suspension_chargeperiod') || 1; my $firstremind = $input->param('firstremind'); my $chargeperiod = $input->param('chargeperiod'); -- 2.20.1