From 794bb066aaedbbd73cd66c08744c52f25b725d96 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Fri, 16 Jun 2017 10:01:12 -0400 Subject: [PATCH] Bug 18434 - Followup - same changes for sort and facet fields To test: 1 - Index some stuff with multiple fields defined for sorting i.e. Authorites - make heading sortable - default is 110a and 111a for heading - a record with 111a empty will make the sort field empty 2 - view the record: curl http://localhost:9200/koha_kohadev_authorities/data/30?pretty=true 3 - Note the blank field 4 - Apply patch 5 - Reindex 6 - Fields should be correctly populated Unit tests to follow (once I have the originals working for all) Signed-off-by: Tomas Cohen Arazi Signed-off-by: Jonathan Druart (cherry picked from commit 241be8ff1ae24fff6165c8d6877945637467ee9a) Signed-off-by: Fridolin Somers --- Koha/SearchEngine/Elasticsearch.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Koha/SearchEngine/Elasticsearch.pm b/Koha/SearchEngine/Elasticsearch.pm index b079dcaea5..43bb0a83a8 100644 --- a/Koha/SearchEngine/Elasticsearch.pm +++ b/Koha/SearchEngine/Elasticsearch.pm @@ -328,7 +328,7 @@ sub get_fixer_rules { $options = '-split => 1' unless $marc_field =~ m|_/| || $type eq 'sum'; push @rules, "marc_map('$marc_field','${name}.\$append', $options)"; if ($facet) { - push @rules, "marc_map('$marc_field','${name}__facet', $options)"; + push @rules, "marc_map('$marc_field','${name}__facet.\$append', $options)"; } if ($suggestible) { push @rules, @@ -351,7 +351,7 @@ sub get_fixer_rules { # to do the default thing, which is make it sortable. if ($self->sort_fields()->{$name}) { if ($sort || !defined $sort) { - push @rules, "marc_map('$marc_field','${name}__sort', $options)"; + push @rules, "marc_map('$marc_field','${name}__sort.\$append', $options)"; } } } -- 2.39.5