Bug 4064 - Uninitialized variable errors in opac-search.pl Merely optimized two if...
authorMark Tompsett <mtompset@hotmail.com>
Sat, 30 Jun 2012 05:45:25 +0000 (13:45 +0800)
committerPaul Poulain <paul.poulain@biblibre.com>
Mon, 3 Sep 2012 13:46:49 +0000 (15:46 +0200)
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
opac/opac-search.pl

index 82b35d2..46cf105 100755 (executable)
@@ -619,7 +619,7 @@ for (my $i=0;$i<@servers;$i++) {
 
             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_history || "unknown",
                                 "query_cgi"  => $query_cgi_history  || "unknown",
@@ -641,7 +641,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_history, $query_cgi_history, $total);
                     $template->param(ShowOpacRecentSearchLink => 1);
                 }