From 7df185c7838f415b40de2e8f3ec918e96f8481cb Mon Sep 17 00:00:00 2001 From: Alex Arnaud Date: Thu, 9 May 2019 12:39:51 +0000 Subject: [PATCH] Bug 22874: Use false instead of 0 value for limiting on availability MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Test plan - Set up a elasticsearch 6 instance to work with Koha, - you may need to make koha works with ES 6 (see bug 20589), - make a search and limit it to available items only, => no result - Apply this patch, - make a search and limit it to available items only, - you should get some results - Do the same with ES 5.x Signed-off-by: Séverine QUEUNE Signed-off-by: Nick Clemens Signed-off-by: Martin Renvoize --- Koha/SearchEngine/Elasticsearch/QueryBuilder.pm | 2 +- t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm index 3e63086730..b418739cdf 100644 --- a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm +++ b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm @@ -965,7 +965,7 @@ sub _fix_limit_special_cases { push @new_lim, "copydate:$date"; } elsif ( $l =~ /^available$/ ) { - push @new_lim, 'onloan:0'; + push @new_lim, 'onloan:false'; } else { push @new_lim, $l; diff --git a/t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t b/t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t index fd983498aa..0f8ed2e9a4 100644 --- a/t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t +++ b/t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t @@ -346,10 +346,10 @@ subtest 'build_query tests' => sub { "query of just '*' is unaltered when QueryAutoTruncate is enabled" ); - ( undef, $query ) = $qb->build_query_compat( undef, ['"donald duck"'] ); + ( undef, $query ) = $qb->build_query_compat( undef, ['"donald duck"'], undef, ['available'] ); is( $query->{query}{query_string}{query}, - '("donald duck")', + '("donald duck") AND onloan:false', "query with quotes is unaltered when QueryAutoTruncate is enabled" ); -- 2.39.5