Attachment 88106 Details for Bug 14715 – Bug 14715: (QA Follow up) Fix results not showing

[patch] Bug 14715: (QA Follow up) Fix results not showing

Bug-14715-QA-Follow-up-Fix-results-not-showing.patch (text/plain), 1.97 KB, created by Liz Rea on 2019-04-16 19:31:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Liz Rea

Created: 2019-04-16 19:31:44 UTC

Size: 1.97 KB

patch

obsolete

>From d2cb2df9f223ef8e79aebdfcbd50fc526df39fd5 Mon Sep 17 00:00:00 2001
>From: Aleisha Amohia <aleishaamohia@hotmail.com>
>Date: Tue, 28 Aug 2018 21:41:26 +0000
>Subject: [PATCH] Bug 14715: (QA Follow up) Fix results not showing
>
>This fixes the display issue for me. Can someone please test?
>
>Signed-off-by: Cab Vinton <bibliwho@gmail.com>
>
>Signed-off-by: Liz Rea <wizzyrea@gmail.com>
>---
> catalogue/search.pl | 5 +----
> opac/opac-search.pl | 5 +----
> 2 files changed, 2 insertions(+), 8 deletions(-)
>
>diff --git a/catalogue/search.pl b/catalogue/search.pl
>index c27e04be03..e0939931b9 100755
>--- a/catalogue/search.pl
>+++ b/catalogue/search.pl
>@@ -453,10 +453,7 @@ for (my $ii = 0; $ii < @operands; ++$ii)
> # Params that can only have one value
> my $scan = $params->{'scan'};
> my $count = C4::Context->preference('numSearchResults') || 20;
>-my $results_per_page = $count;
>-if ( C4::Context->preference('numSearchResultsDropdown') ){
>-    $results_per_page = $params->{'results_per_page'} || $cgi->cookie("results_per_page");
>-}
>+my $results_per_page = $params->{'results_per_page'} || $cgi->cookie("results_per_page") || $count;
> my $page = $cgi->param('page') || 1;
> my $offset = $params->{'offset'} || 0;
> $offset = 0 if $offset < 0;
>diff --git a/opac/opac-search.pl b/opac/opac-search.pl
>index 27d80ba679..efd425b9f0 100755
>--- a/opac/opac-search.pl
>+++ b/opac/opac-search.pl
>@@ -523,10 +523,7 @@ if ($params->{'limit-yr'}) {
> my $scan = $params->{'scan'};
> my $count = C4::Context->preference('OPACnumSearchResults') || 20;
> my $countRSS         = C4::Context->preference('numSearchRSSResults') || 50;
>-my $results_per_page = $count;
>-if ( C4::Context->preference('OPACnumSearchResultsDropdown') ){
>-    $results_per_page = $params->{'results_per_page'} || $cgi->cookie("results_per_page");
>-}
>+my $results_per_page = $params->{'results_per_page'} || $cgi->cookie("results_per_page") || $count;
> my $page = $cgi->param('page') || 1;
> my $offset = $params->{'offset'} || 0;
> $offset = 0 if $offset < 0;
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 14715: 41949 | 42003 | 42742 | 42743 | 56002 | 56003 | 56004 | 56005 | 56575 | 56576 | 56577 | 56610 | 56611 | 56612 | 57870 | 61721 | 64250 | 67912 | 67913 | 68000 | 68001 | 68305 | 68327 | 75938 | 75939 | 75940 | 75941 | 78230 | 78298 | 78299 | 78300 | 78301 | 78302 | 79796 | 79797 | 79988 | 81203 | 88097 | 88102 | 88103 | 88104 | 88105 | 88106 | 88107 | 88108 | 88109 | 88126 | 88195 | 88831 | 88832 | 88833 | 88834 | 88835 | 88836 | 88837 | 88838 | 88839 | 95038 | 97004 | 98365 | 99607 | 99677 | 99684 | 99685 | 101775 | 101776 | 101777 | 101778