Attachment 10836 Details for Bug 8381 – [SIGNED-OFF] Bug 8381 - Paging broken in OPAC search results

[patch] [SIGNED-OFF] Bug 8381 - Paging broken in OPAC search results

SIGNED-OFF-Bug-8381---Paging-broken-in-OPAC-search.patch (text/plain), 1.90 KB, created by Katrin Fischer on 2012-07-14 09:39:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2012-07-14 09:39:04 UTC

Size: 1.90 KB

patch

obsolete

>From 11e6164790e09eed59f489f67f449b1ceceff986 Mon Sep 17 00:00:00 2001
>From: Alex Arnaud <alex.arnaud@biblibre.com>
>Date: Fri, 13 Jul 2012 11:13:47 +0200
>Subject: [PATCH] [SIGNED-OFF] Bug 8381 - Paging broken in OPAC search results
>
>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 |   12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
>diff --git a/opac/opac-search.pl b/opac/opac-search.pl
>index e39e547..5c6e85b 100755
>--- a/opac/opac-search.pl
>+++ b/opac/opac-search.pl
>@@ -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
>                               };
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 8381: 10809 | 10836 | 10837 | 10867