From cfb52579ec2f8859226e0e40f66d5a6d7a2ed73f Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Fri, 16 Jun 2017 09:53:39 -0400 Subject: [PATCH] Bug 18624: Run time errors when searching authorities with Elastic search 5.3 It seems we have a syntax mismatch - any vs all all seems to the be list we expect so updating code that way To test: 1 - Enabled Elasticsearch 2 - Index some authorities 3 - Perform a 'Search entire record' search 4 - Internal server error ( Invalid marclist field provided: all at /usr/local/koha/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm line 433. ) 5 - Run: $ sudo koha-shell kohadev k$ cd kohaclone k$ prove t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t => FAIL: Tests fail because 'any is used' 6 - Apply patch 7 - Search should work 8 - Run: k$ prove t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t => SUCCESS: Tests pass! 9 Sign off :-D Signed-off-by: Tomas Cohen Arazi Edited the test plan so it mentions the new tests Signed-off-by: Marcel de Rooy Signed-off-by: Jonathan Druart --- Koha/SearchEngine/Elasticsearch/QueryBuilder.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm index 2fe923600a..4169211857 100644 --- a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm +++ b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm @@ -416,7 +416,7 @@ our $koha_to_index_name = { 'match-heading' => 'Match-heading', 'see-from' => 'Match-heading-see-from', thesaurus => 'Subject-heading-thesaurus', - any => '' + all => '' }; sub build_authorities_query_compat { -- 2.39.5