Attachment 62702 Details for Bug 18434 – Bug 18434 - Elasticsearch indexing broken with newer catmandu version

[patch] Bug 18434 - Elasticsearch indexing broken with newer catmandu version

Bug-18434---Elasticsearch-indexing-broken-with-new.patch (text/plain), 1.57 KB, created by Chris Cormack on 2017-04-25 20:50:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Cormack

Created: 2017-04-25 20:50:05 UTC

Size: 1.57 KB

patch

obsolete

>From 9131cd60650efc5c1065be3b35811bb74c3626b9 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Fri, 14 Apr 2017 10:03:11 -0400
>Subject: [PATCH] Bug 18434 - Elasticsearch indexing broken with newer catmandu
> version
>
>To test:
>1 - Make sure you have latest koha deps, catmandu versions should be:
>    libcatmandu-marc-perl   1.09-1~kohadev1
>    libcatmandu-perl        1.0304-2~kohadev1
>2 - Reindex elastic
>3 - Try searching and likely notice odd results
>4 - Try:
>curl -XGET
>'http://localhost:9200/koha_kohadev_biblios/data/792?pretty=true'
>with a known biblionumber and notice some null fields
>5 - Apply patch
>6 - Reindex
>7 - Note fields are populated and search works as expected
>
>Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
>---
> Koha/SearchEngine/Elasticsearch.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/Koha/SearchEngine/Elasticsearch.pm b/Koha/SearchEngine/Elasticsearch.pm
>index 1fd8bc1465..b079dcaea5 100644
>--- a/Koha/SearchEngine/Elasticsearch.pm
>+++ b/Koha/SearchEngine/Elasticsearch.pm
>@@ -326,7 +326,7 @@ sub get_fixer_rules {
>             # The split makes everything into nested arrays, but that's not
>             # really a big deal, ES doesn't mind.
>             $options = '-split => 1' unless $marc_field =~ m|_/| || $type eq 'sum';
>-            push @rules, "marc_map('$marc_field','${name}', $options)";
>+            push @rules, "marc_map('$marc_field','${name}.\$append', $options)";
>             if ($facet) {
>                 push @rules, "marc_map('$marc_field','${name}__facet', $options)";
>             }
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 18434: 62171 | 62702 | 63451 | 63452 | 63453 | 63473 | 63477 | 63478 | 63479 | 64386 | 64773 | 64786 | 64787 | 64788 | 64789 | 64790