Attachment 133515 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.29 KB, created by Aleisha Amohia on 2022-04-21 04:20:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Aleisha Amohia

Created: 2022-04-21 04:20:32 UTC

Size: 1.29 KB

patch

obsolete

>From 6fc8fda2bce32cc449bb88c5dc945940c20d4d63 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
>
>Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>
>---
> 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 3e2cb1b784..43ee56684d 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.11.0

Actions: View | Diff | Splinter Review

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