From b3c73cff32c99f12b77a9b110e2a928f5dfd1679 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Tue, 7 Jan 2014 10:19:37 -0700 Subject: [PATCH] Bug 11489: in OPAC search, display "no results" when the only search result is suppressed If hidelostitems is enabled, and the result of an opac search is a single lost item, then the OPAC will display a 404 error, rather than a "no results" screen. Test Plan: 1) Catalog a record/item such that it is the only result for some search e.g. Give it the title 'zxcvb' 2) Enable hidelostitems 3) Mark this item as lost 4) Perform an OPAC search that should result in a redirect to this record 5) Notice you a redirected to a 404 error 6) Apply this patch 7) Repeat step 4 8) Note you new get a "No results found!" page instead Signed-off-by: Christopher Brannon Signed-off-by: Michot Signed-off-by: Katrin Fischer Signed-off-by: Galen Charlton (cherry picked from commit 2277a42c565e18475c49fde031268b8038575e1a) Signed-off-by: Fridolin Somers --- opac/opac-search.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/opac/opac-search.pl b/opac/opac-search.pl index 617d67ee1b..a8eb7d54ea 100755 --- a/opac/opac-search.pl +++ b/opac/opac-search.pl @@ -556,6 +556,7 @@ for (my $i=0;$i<@servers;$i++) { @newresults = searchResults('opac', $query_desc, $hits, $results_per_page, $offset, $scan, $results_hashref->{$server}->{"RECORDS"}); } + $hits = 0 unless @newresults; foreach my $res (@newresults) { @@ -610,7 +611,7 @@ for (my $i=0;$i<@servers;$i++) { } if ($results_hashref->{$server}->{"hits"}){ - $total = $total + $results_hashref->{$server}->{"hits"}; + $total = $total + $hits; } # Opac search history -- 2.39.5