Bugzilla – Attachment 73866 Details for
Bug 19581
Elasticsearch - Catmandu split option adds extra null fields to indexes
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 19581: Remove unused comment and line
Bug-19581-Remove-unused-comment-and-line.patch (text/plain), 1.82 KB, created by
Julian Maurice
on 2018-04-09 12:47:15 UTC
(
hide
)
Description:
Bug 19581: Remove unused comment and line
Filename:
MIME Type:
Creator:
Julian Maurice
Created:
2018-04-09 12:47:15 UTC
Size:
1.82 KB
patch
obsolete
>From 314f3a719cf18677b212fb962f27e58f2b2690b3 Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Tue, 13 Mar 2018 09:32:04 +0000 >Subject: [PATCH] Bug 19581: Remove unused comment and line > >Signed-off-by: Julian Maurice <julian.maurice@biblibre.com> >--- > 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 d904bcf710..fe2e0ea17c 100644 >--- a/Koha/SearchEngine/Elasticsearch.pm >+++ b/Koha/SearchEngine/Elasticsearch.pm >@@ -323,20 +323,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.14.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 19581
:
68947
|
68948
|
68949
|
69092
|
69093
|
72745
|
72767
|
72997
|
73864
|
73865
| 73866 |
73867
|
73896