Attachment 78560 Details for Bug 21331 – Bug 21331: Elasticsearch - fix indexing of concatenated subfields

[patch] Bug 21331: Elasticsearch - fix indexing of concatenated subfields

Bug-21331-Elasticsearch---fix-indexing-of-concaten.patch (text/plain), 2.00 KB, created by Alex Arnaud on 2018-09-11 15:51:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Arnaud

Created: 2018-09-11 15:51:31 UTC

Size: 2.00 KB

patch

obsolete

>From 397287238c8fb351fc90e46161a6092ae6f71401 Mon Sep 17 00:00:00 2001
>From: Alex Arnaud <alex.arnaud@biblibre.com>
>Date: Tue, 11 Sep 2018 15:47:09 +0000
>Subject: [PATCH] Bug 21331: Elasticsearch - fix indexing of concatenated
> subfields
>
>---
> Koha/SearchEngine/Elasticsearch.pm | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
>diff --git a/Koha/SearchEngine/Elasticsearch.pm b/Koha/SearchEngine/Elasticsearch.pm
>index 9c114fb..58c5e62 100644
>--- a/Koha/SearchEngine/Elasticsearch.pm
>+++ b/Koha/SearchEngine/Elasticsearch.pm
>@@ -292,16 +292,15 @@ sub get_fixer_rules {
>         sub {
>             my ( $name, $type, $facet, $suggestible, $sort, $marc_type, $marc_field ) = @_;
>             return if $marc_type ne $marcflavour;
>-            my $options ='';
> 
>-            push @rules, "marc_map('$marc_field','${name}.\$append', $options)";
>+            push @rules, "marc_map('$marc_field','${name}.\$append', pluck:0, join:' ')";
>             if ($facet) {
>-                push @rules, "marc_map('$marc_field','${name}__facet.\$append', $options)";
>+                push @rules, "marc_map('$marc_field','${name}__facet.\$append', pluck:0, join:' ')";
>             }
>             if ($suggestible) {
>                 push @rules,
>                     #"marc_map('$marc_field','${name}__suggestion.input.\$append', '')"; #must not have nested data structures in .input
>-                    "marc_map('$marc_field','${name}__suggestion.input.\$append')";
>+                    "marc_map('$marc_field','${name}__suggestion.input.\$append', pluck:0, join:' ')";
>             }
>             if ( $type eq 'boolean' ) {
> 
>@@ -315,7 +314,7 @@ sub get_fixer_rules {
>             }
>             if ($self->sort_fields()->{$name}) {
>                 if ($sort || !defined $sort) {
>-                    push @rules, "marc_map('$marc_field','${name}__sort.\$append', $options)";
>+                    push @rules, "marc_map('$marc_field','${name}__sort.\$append', pluck:0, join:' ')";
>                 }
>             }
>         }
>-- 
>2.7.4

Actions: View | Diff | Splinter Review

Attachments on bug 21331: 78560 | 80217