From ce69b1529d4bfb9fd05649e380de91986d54c4c3 Mon Sep 17 00:00:00 2001 From: Mark Tompsett Date: Fri, 29 Jun 2012 23:13:57 +0800 Subject: [PATCH] Bug 8175 - Missing defined check in catalogue/details.pl Actually, just dropping the "ne ''" was better. In some cases $item->{'materials'} isn't defined. Any value is true, so no need for comparison. Signed-off-by: Chris Cormack --- catalogue/detail.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/catalogue/detail.pl b/catalogue/detail.pl index d110740..97ed167 100755 --- a/catalogue/detail.pl +++ b/catalogue/detail.pl @@ -261,7 +261,7 @@ foreach my $item (@items) { $analytics_flag=1; $item->{countanalytics} = $countanalytics; } - if ($item->{'materials'} ne ''){ + if ($item->{'materials'}){ $materials_flag = 1; } push @itemloop, $item; -- 1.7.10