Bugzilla – Attachment 63453 Details for
Bug 18434
Elasticsearch indexing broken with newer catmandu version
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 18434: (followup) Remove unneeded comma in fixer rules
Bug-18434-followup-Remove-unneeded-comma-in-fixer-.patch (text/plain), 1.79 KB, created by
Tomás Cohen Arazi (tcohen)
on 2017-05-12 18:47:19 UTC
(
hide
)
Description:
Bug 18434: (followup) Remove unneeded comma in fixer rules
Filename:
MIME Type:
Creator:
Tomás Cohen Arazi (tcohen)
Created:
2017-05-12 18:47:19 UTC
Size:
1.79 KB
patch
obsolete
>From a7036011ce6686a12bc604586ec776f9771315f9 Mon Sep 17 00:00:00 2001 >From: Tomas Cohen Arazi <tomascohen@theke.io> >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 <tomascohen@theke.io> >--- > 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
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 18434
:
62171
|
62702
|
63451
|
63452
|
63453
|
63473
|
63477
|
63478
|
63479
|
64386
|
64773
|
64786
|
64787
|
64788
|
64789
|
64790