Bug 33569: Fix sort by relevance in catalog search
authorFridolin Somers <fridolin.somers@biblibre.com>
Thu, 20 Apr 2023 06:58:48 +0000 (20:58 -1000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Tue, 9 May 2023 14:46:47 +0000 (11:46 -0300)
commit3bb7ea2dab03863e827fbb21060ac98a16dcbf4f
tree846bb7fb822ca050f77b699bf185192e0c1c63e4
parent7323a56823c0b23faddd964206c8fc58431b769d
Bug 33569: Fix sort by relevance in catalog search

When ordering catalog search by relevance from advanced search, this is not visible in sort order combobox in staff interface.

This is because sort by relevance is a special case not using sort
order.
This patch add a new method C4::Context->default_catalog_sort_by to use
in both staff interface and OPAC.

Test plan :
1.0) On staff interface
1.1) Define system preferences defaultSortField = 'relevance' and defaultSortOrder = 'descending'
1.2) Perform a catalog search from main page on any term, ie 'perl'
=> You see in sort combobox 'relevance' selected (with a check icon)
1.3) Perform an advanced search on any term, ie 'perl', with sort by relevance
=> Without patch : you see in sort combobox 'relevance' not selected (no check icon)
=> With patch : you see in sort combobox 'relevance' selected (with a check icon)
2.0) On OPAC
2.1) Define system preferences OPACdefaultSortField = 'relevance' and OPACdefaultSortOrder = 'descending'
2.2) Perform a catalog search from main page on any term, ie 'perl'
=> You see in sort combobox 'relevance' selected
3.0) Run prove t/Context.t

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
C4/Context.pm
catalogue/search.pl
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt
opac/opac-detail.pl
opac/opac-search.pl
t/Context.t