From cf29caf39fed2d81262735fe7893a1ac3a8d59b1 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Thu, 21 Oct 2021 10:26:12 +0000
Subject: [PATCH] Bug 29284: (follow-up) Improve error logging to prevent warns

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
---
 Koha/SearchEngine/Elasticsearch/QueryBuilder.pm | 1 +
 catalogue/detail.pl                             | 7 +++++--
 opac/opac-detail.pl                             | 7 +++++--
 3 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
index 01b62be4e8..9627a3f6ce 100644
--- a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
+++ b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
@@ -986,6 +986,7 @@ sub clean_search_term {
     $term =~ s/((?<!\\)(?:[\\]{2})*:[^:\s]+(?<!\\)(?:[\\]{2})*)(?=:)/$1\\/g;
 
     # screen all exclamation signs that either are the last symbol or have white space after them
+    # or are followed by close parentheses
     $term =~ s/(?:[\s\\]*!\s*)+(\s|$|\))/$1/g;
 
     # screen all brackets with backslash
diff --git a/catalogue/detail.pl b/catalogue/detail.pl
index 4b1ad5c6ea..10352625e5 100755
--- a/catalogue/detail.pl
+++ b/catalogue/detail.pl
@@ -146,12 +146,15 @@ if ( $xslfile ) {
 
     };
     if ($err || $@){
-        warn "Warning from simple_search_compat: $err.$@";
+        my $error = q{};
+        $error .= $err if $err;
+        $error .= $@ if $@;
+        warn "Warning from simple_search_compat: $error";
         $template->param( analytics_error => 1 );
     }
 
     my $variables = {
-        show_analytics_link => $count > 0 ? 1 : 0
+        show_analytics_link => defined $count && $count > 0 ? 1 : 0
     };
 
     $template->param(
diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
index e374aa7d2e..3889ab5406 100755
--- a/opac/opac-detail.pl
+++ b/opac/opac-detail.pl
@@ -202,12 +202,15 @@ if ( $xslfile ) {
           $searcher->simple_search_compat( $query, 0, 0 );
     };
     if ($err || $@){
-        warn "Warning from simple_search_compat: $err.$@";
+        my $error = q{};
+        $error .= $err if $err;
+        $error .= $@ if $@;
+        warn "Warning from simple_search_compat: $error";
     }
 
     my $variables = {
         anonymous_session   => ($borrowernumber) ? 0 : 1,
-        show_analytics_link => $count > 0 ? 1 : 0
+        show_analytics_link => defined $count && $count > 0 ? 1 : 0
     };
 
     my @plugin_responses = Koha::Plugins->call(
-- 
2.39.5