From 21148a4242c1da4e2caf04e1cf9e2722c9e43c5c Mon Sep 17 00:00:00 2001 From: Petro Vashchuk Date: Mon, 26 Jul 2021 12:27:40 +0300 Subject: [PATCH] Bug 28736: fix requirements for the correct error message to show up There is existing error message in the code stating: "Unable to understand your search query, please rephrase and try again." which fits perfectly but because it looks for "ParseException" in the warning output it doesn't show up on this page as it's actually "parse_exception". This patch makes that it's also checked if "parse_exception" is present in the warning output. To reproduce: 1) using ES search for something like "// ^ ! { } [ ] .. , <>" that will for sure break the syntax of ES. 2) after the search query fails note that the error is "Unable to perform your search. Please try again." 3) apply the patch 4) search for the same thing again 5) error message should be "Unable to understand your search query, please rephrase and try again." now. Signed-off-by: David Nind --- Koha/SearchEngine/Elasticsearch.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Koha/SearchEngine/Elasticsearch.pm b/Koha/SearchEngine/Elasticsearch.pm index 8950f8fb9e..803f152487 100644 --- a/Koha/SearchEngine/Elasticsearch.pm +++ b/Koha/SearchEngine/Elasticsearch.pm @@ -1218,7 +1218,7 @@ sub process_error { warn $msg; # simple logging # This is super-primitive - return "Unable to understand your search query, please rephrase and try again.\n" if $msg =~ /ParseException/; + return "Unable to understand your search query, please rephrase and try again.\n" if $msg =~ /ParseException|parse_exception/; return "Unable to perform your search. Please try again.\n"; } -- 2.20.1