From 728c077a6039c78920eedf21a0d78abd5f51169c Mon Sep 17 00:00:00 2001 From: Mark Tompsett Date: Tue, 4 Sep 2012 08:15:19 +0800 Subject: [PATCH] Bug 4064 - Uninitialized variable errors in opac-search.pl In 3.6.3 the error was triggered. It was already fixed with some particularly ugly logic: if (($params->{'offset'}||'') eq '') { Having read through the code, $offset is a better choice and produces prettier, clearer code: if (!$offset) { Signed-off-by: Chris Cormack (cherry picked from commit 86a1246fcef0456758c7c61985d427202bbfb860) Signed-off-by: Jared Camins-Esakov --- opac/opac-search.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/opac/opac-search.pl b/opac/opac-search.pl index d7b34a4995..266e3d73a0 100755 --- a/opac/opac-search.pl +++ b/opac/opac-search.pl @@ -537,7 +537,7 @@ for (my $i=0;$i<@servers;$i++) { # Adding the new search if needed if (!$borrowernumber || $borrowernumber eq '') { # To a cookie (the user is not logged in) - if (($params->{'offset'}||'') eq '') { + if (!$offset) { push @recentSearches, { "query_desc" => $query_desc || "unknown", "query_cgi" => $query_cgi || "unknown", @@ -559,7 +559,7 @@ for (my $i=0;$i<@servers;$i++) { } else { # To the session (the user is logged in) - if (($params->{'offset'}||'') eq '') { + if (!$offset) { AddSearchHistory($borrowernumber, $cgi->cookie("CGISESSID"), $query_desc, $query_cgi, $total); $template->param(ShowOpacRecentSearchLink => 1); } -- 2.39.5