From 5be6dc422bcc9474b131868d473da51971e7e3c0 Mon Sep 17 00:00:00 2001
From: Colin Campbell <colin.campbell@ptfs-europe.com>
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 <gmcharlt@gmail.com>
---
 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