Bug 25701: (bug 14419 follow-up) Always display facet groups in the same order
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 10 Jun 2020 10:03:20 +0000 (12:03 +0200)
committerVictor Grousset/tuxayo <victor@tuxayo.net>
Fri, 19 Jun 2020 03:56:11 +0000 (05:56 +0200)
commit014efe3dd63d12b6eba88126bb5ea31a07897643
treedc91c7af309c8dd7bdfb900e33a9ed8a9d75a491
parent879c34aef6e3c720655531945b9f8c8544fd3919
Bug 25701: (bug 14419 follow-up) Always display facet groups in the same order

It seems that this regression comes from bug 14419, but I have not found
a logic reason behind that.
This patch restores the behaviour we always had: facet groups must be
displayed in a given order: the Zebra index (au, ccode, holdingbranch,
etc.)

Test plan:
Apply this patch, restart all and confirm that the facets are not moving
up and down when you refresh your search result page.

QA note:
I think the following line must be removed
@facets_loop = sort {defined $a->{expand} && defined $b->{expand} && $a->{expand} cmp $b->{expand}} @facets_loop;

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit d6696944267ca1afd7682193ddf3680a905a773d)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 91ad11021609bc15e9099f7fe9d4afeb69786647)

Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>

(cherry picked from commit dcfb394e03e3ccb56b2f6bc21e481cfd001548a1)
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
C4/Search.pm