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
---
acqui/basket.pl | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/acqui/basket.pl b/acqui/basket.pl
index 7c2456a..9381ae6 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;
--
1.6.6.1
_______________________________________________
Koha-patches mailing list
Koha-patches@lists.koha.org
http://lists.koha.org/mailman/listinfo/koha-patches
