From 5be6dc422bcc9474b131868d473da51971e7e3c0 Mon Sep 17 00:00:00 2001 From: Colin Campbell Date: Wed, 19 May 2010 15:00:09 +0100 Subject: [PATCH] Remove generation of an unnecessary warning We've checked the return for definedness and assigned an empty string to it ao we don't want to make it undef again This is a good argument that cascading if-elses expecially with long unstructured elses are a bad idea Signed-off-by: Galen Charlton --- acqui/basket.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/acqui/basket.pl b/acqui/basket.pl index 7c2456a961..9381ae6abc 100755 --- a/acqui/basket.pl +++ b/acqui/basket.pl @@ -172,7 +172,7 @@ if ( $op eq 'delete_confirm' ) { basketgroupname => $basket->{'basketname'}); } -} elsif ($query->param('op') eq 'reopen') { +} elsif ($op eq 'reopen') { my $basket; $basket->{basketno} = $query->param('basketno'); $basket->{closedate} = undef; -- 2.39.5