From a7036011ce6686a12bc604586ec776f9771315f9 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Fri, 12 May 2017 15:45:41 -0300 Subject: [PATCH] Bug 18434: (followup) Remove unneeded comma in fixer rules Signed-off-by: Tomas Cohen Arazi --- Koha/SearchEngine/Elasticsearch.pm | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Koha/SearchEngine/Elasticsearch.pm b/Koha/SearchEngine/Elasticsearch.pm index b079dca..5b75dc1 100644 --- a/Koha/SearchEngine/Elasticsearch.pm +++ b/Koha/SearchEngine/Elasticsearch.pm @@ -325,10 +325,10 @@ sub get_fixer_rules { # selects a range # 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}.\$append', $options)"; + $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' $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' $options)"; } } } -- 2.7.4