Koha-clms/Koha/SearchEngine
Nick Clemens 9418010d63 Bug 19563: (follow-up) Restore checking sort variable
This code seems a bit odd, $self->sort_fields()->{$name} is checking if
there is any mapping in ES for a field, so if one marc field in an index
is marked as sortable this will exists. We need to check the individual
marc field to see if it should be added to the sort index here.

If you apply the first patch, reindex, and view a record in es:
curl -XGET 'localhost:9200/koha_kohadev_biblios/data/19/?pretty' | grep
-A 10 author__sort

You will see that 245$c is included in the record without that
additional check

Apply this, reindex again, and that field should not be included

Signed-off-by: David Bourgault <david.bourgault@inlibro.com>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
2017-12-15 11:28:12 +00:00
..
Elasticsearch Bug 19559: Add '-' to list of characters we don't split search terms on 2017-11-26 12:59:46 -03:00
Zebra Bug 16660: (followup) Unit tests 2017-10-27 14:09:02 -03:00
Elasticsearch.pm Bug 19563: (follow-up) Restore checking sort variable 2017-12-15 11:28:12 +00:00
QueryBuilder.pm Bug 12478: allow more granular sorting configuration 2016-04-26 20:20:08 +00:00
Search.pm Bug 9988: Add extract_biblionumber to Koha/SearchEngine 2017-04-13 08:53:46 -04:00