From c5865d6bcabe6f57f6a3b3d217cf43481c3ec363 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Thu, 5 Sep 2013 15:49:55 +0100 Subject: [PATCH] Bug 10829: fix browsing across pages breaks for OPAC bibs-linked-to-authority searches Test plan 1. Do an authority search in the OPAC (A search for 'an:49' perhaps.. that should bring up results for authority number 49, hopefully there are multiple pages of results, if not try again and find one that gives you multiple pages of results) 2. Click the final result on any page of results (there must be more results to follow for this to work) 3. Try clicking the 'Next' button in the 'Browse results' pane on the right of the page. 4. It should take you to the next result in the search list (i.e the first result from the next page of results) - Double check it is correct, we found that it was often not. 5. Apply patch and run same process, this time you should get the correct paging. Signed-off-by: Koha Team Amu Signed-off-by: Jonathan Druart I reproduce the issue and I confirm this patch fixes it! Signed-off-by: Galen Charlton (cherry picked from commit ded9896ec28c404ab27d5bef54eabb570c636841) Signed-off-by: Tomas Cohen Arazi --- opac/opac-detail.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl index c47eb31bb3..8bc66938e4 100755 --- a/opac/opac-detail.pl +++ b/opac/opac-detail.pl @@ -142,6 +142,7 @@ if ($session->param('busc')) { my $expanded_facet = $arrParamsBusc->{'expand'}; my $branches = GetBranches(); + my $itemtypes = GetItemTypes; my @servers; @servers = @{$arrParamsBusc->{'server'}} if $arrParamsBusc->{'server'}; @servers = ("biblioserver") unless (@servers); @@ -152,7 +153,7 @@ if ($session->param('busc')) { $sort_by[0] = $default_sort_by if !$sort_by[0] && defined($default_sort_by); my ($error, $results_hashref, $facets); eval { - ($error, $results_hashref, $facets) = getRecords($arrParamsBusc->{'query'},$arrParamsBusc->{'simple_query'},\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$arrParamsBusc->{'query_type'},$arrParamsBusc->{'scan'}); + ($error, $results_hashref, $facets) = getRecords($arrParamsBusc->{'query'},$arrParamsBusc->{'simple_query'},\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$itemtypes,$arrParamsBusc->{'query_type'},$arrParamsBusc->{'scan'}); }; my $hits; my @newresults; -- 2.39.5