Attachment 125132 Details for Bug 29077 – Bug 29077: Remove blank indexes before convert

[patch] Bug 29077: Remove blank indexes before convert

Bug-29077-Remove-blank-indexes-before-convert.patch (text/plain), 1.23 KB, created by Nick Clemens (kidclamp) on 2021-09-22 12:58:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2021-09-22 12:58:14 UTC

Size: 1.23 KB

patch

obsolete

>From 59e16074fa74efa7b96570d34d1ae6b1bb13a5cf Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Wed, 22 Sep 2021 12:55:19 +0000
>Subject: [PATCH] Bug 29077: Remove blank indexes before convert
>
>To test:
>1 - Be using Elasticsearch
>2 - Search on OPAC with dropdown set to 'Library catalog'
>3 - Warns in plack-opac-error.log
>4 - Apply patch
>5 - Repeat
>6 - No warns
>7 - Search returns correct results
>---
> Koha/SearchEngine/Elasticsearch/QueryBuilder.pm | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
>index 58d5155ab4..b1aecd27e1 100644
>--- a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
>+++ b/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
>@@ -714,6 +714,8 @@ sub _convert_index_fields {
>     my %index_type_convert =
>       ( __default => undef, phr => 'phrase', rtrn => 'right-truncate', 'st-year' => 'st-year' );
> 
>+    @indexes = grep(/.+/, @indexes); # Remove any blank indexes, i.e. keyword
>+
>     # Convert according to our table, drop anything that doesn't convert.
>     # If a field starts with mc- we save it as it's used (and removed) later
>     # when joining things, to indicate we make it an 'OR' join.
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 29077: 125132 | 133515 | 133886 | 134006