Attachment 93359 Details for Bug 22874 – Bug 22874 - use false instaed of 0 value for limiting on availability

[patch] Bug 22874 - use false instaed of 0 value for limiting on availability

Bug-22874---use-false-instaed-of-0-value-for-limit.patch (text/plain), 2.33 KB, created by Biblibre Sandboxes on 2019-10-01 12:14:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Biblibre Sandboxes

Created: 2019-10-01 12:14:38 UTC

Size: 2.33 KB

patch

obsolete

>From 2a81bd9cc2da92d8b4e057c9c0e1f55fa8d9fe0c Mon Sep 17 00:00:00 2001
>From: Alex Arnaud <alex.arnaud@biblibre.com>
>Date: Thu, 9 May 2019 12:39:51 +0000
>Subject: [PATCH] Bug 22874 - use false instaed of 0 value for limiting on
> availability
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>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 <severine.queune@bulac.fr>
>Signed-off-by: SÃ©verine QUEUNE <severine.queune@bulac.fr>
>---
> 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 ce59ff0..859f538 100644
>--- a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
>+++ b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
>@@ -881,7 +881,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 fe69eb3..b025e81 100644
>--- a/t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t
>+++ b/t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t
>@@ -331,10 +331,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.7.4

Actions: View | Diff | Splinter Review

Attachments on bug 22874: 89506 | 93359 | 94792