Bug 23089: Fix sorting on sum and regular fields
authorNick <nick@bywatersolutions.com>
Thu, 10 Oct 2019 16:06:08 +0000 (16:06 +0000)
committerLucas Gass <lucas@bywatersolutions.com>
Tue, 17 Dec 2019 22:46:18 +0000 (22:46 +0000)
commit1044eb1f30042a6b869455f1107cfb7cbc9c6e4a
tree323ed674f48b6771d949fb2df1874c49da4ed847
parent8c175ec3e4cd330a4cb59d3cbeebb14ba366dcaa
Bug 23089: Fix sorting on sum and regular fields

As of bug 20589 we no longer analyze sort fields and so we no longer need to append ".phrase"
to our sort in searches.

Additionally, sort fields based on 'sum' should also use sum in building the value to sort on

To test:
0 - Be using ES
1 - Find the most circulated item in your collection
2 - Search for '*'
3 - Sort by popularity DESC
4 - Note that item is not first
5 - Try to sort by anything but relevancy, it fails
6 - Apply patch
7 - Redo searches and sorts
8 - Things should now work as expected

Signed-off-by: Ere Maijala <ere.maijala@helsinki.fi>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 6a2ba8b6b23f482dcf8bbe53668d93519a11edf5)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Koha/SearchEngine/Elasticsearch.pm
Koha/SearchEngine/Elasticsearch/QueryBuilder.pm
admin/searchengine/elasticsearch/field_config.yaml