From e14d628790a55017832a0946b9d21556f3269ad9 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. Content-Type: text/plain; charset="utf-8" --- 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.9.5