Bug 8381 - Paging broken in OPAC search results
authorAlex Arnaud <alex.arnaud@biblibre.com>
Fri, 13 Jul 2012 09:13:47 +0000 (11:13 +0200)
committerPaul Poulain <paul.poulain@biblibre.com>
Tue, 17 Jul 2012 12:59:26 +0000 (14:59 +0200)
Fix also freezing for some browser when performing a search query.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Checked paging (previous, next and navigating to page numbers) works.
Browser performance is fixed too.

opac/opac-search.pl

index e39e547..5c6e85b 100755 (executable)
@@ -602,17 +602,17 @@ for (my $i=0;$i<@servers;$i++) {
 
             # Adding the new search if needed
             my $path_info = $cgi->url(-path_info=>1);
-            $query_cgi = $cgi->url(-query=>1);
-            $query_cgi =~ s/^$path_info\?//;
-            $query_cgi =~ s/;/&/g;
-            $query_desc .= ", $limit_desc";
+            my $query_cgi_history = $cgi->url(-query=>1);
+            $query_cgi_history =~ s/^$path_info\?//;
+            $query_cgi_history =~ s/;/&/g;
+            my $query_desc_history = "$query_desc, $limit_desc";
 
             if (!$borrowernumber || $borrowernumber eq '') {
                 # To a cookie (the user is not logged in)
                 if (($params->{'offset'}||'') eq '') {
                     push @recentSearches, {
-                                "query_desc" => $query_desc || "unknown",
-                                "query_cgi"  => $query_cgi  || "unknown",
+                                "query_desc" => $query_desc_history || "unknown",
+                                "query_cgi"  => $query_cgi_history  || "unknown",
                                 "time"       => time(),
                                 "total"      => $total
                               };