From 5308cfd0471a71780ca9b89167eea25d3922d2b8 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Tue, 13 Mar 2018 09:32:04 +0000 Subject: [PATCH] Bug 19581: Remove unused comment and line Signed-off-by: Julian Maurice Signed-off-by: Jonathan Druart Signed-off-by: Nick Clemens --- Koha/SearchEngine/Elasticsearch.pm | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/Koha/SearchEngine/Elasticsearch.pm b/Koha/SearchEngine/Elasticsearch.pm index e8b7ac4c1b..1b6a29544a 100644 --- a/Koha/SearchEngine/Elasticsearch.pm +++ b/Koha/SearchEngine/Elasticsearch.pm @@ -320,20 +320,15 @@ sub get_fixer_rules { sub { my ( $name, $type, $facet, $suggestible, $sort, $marc_type, $marc_field ) = @_; return if $marc_type ne $marcflavour; - my $options = ''; + my $options =''; - # There's a bug when using 'split' with something that - # selects a range - # The split makes everything into nested arrays, but that's not - # really a big deal, ES doesn't mind. - $options = '' 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.\$append', $options)"; } if ($suggestible) { push @rules, - #"marc_map('$marc_field','${name}__suggestion.input.\$append', $options)"; #must not have nested data structures in .input + #"marc_map('$marc_field','${name}__suggestion.input.\$append', '')"; #must not have nested data structures in .input "marc_map('$marc_field','${name}__suggestion.input.\$append')"; } if ( $type eq 'boolean' ) { -- 2.39.5