Bugzilla – Attachment 75974 Details for
Bug 18213
Add language facets to Elasticsearch
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 18213: (follow-up) Correctly sort facets
Bug-18213-follow-up-Correctly-sort-facets.patch (text/plain), 1.01 KB, created by
Alex Arnaud
on 2018-06-12 14:10:55 UTC
(
hide
)
Description:
Bug 18213: (follow-up) Correctly sort facets
Filename:
MIME Type:
Creator:
Alex Arnaud
Created:
2018-06-12 14:10:55 UTC
Size:
1.01 KB
patch
obsolete
>From afd4194cdab5b19403e4401f94636446a9c5bc38 Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Thu, 15 Mar 2018 12:53:13 +0000 >Subject: [PATCH] Bug 18213: (follow-up) Correctly sort facets > >Previously we sorted alphabetically - that works for 1-9. >With a 10th facet category we should sort numerically > >Signed-off-by: Nicolas Legrand <nicolas.legrand@bulac.fr> >Signed-off-by: Alex Arnaud <alex.arnaud@biblibre.com> >--- > Koha/SearchEngine/Elasticsearch/Search.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/Koha/SearchEngine/Elasticsearch/Search.pm b/Koha/SearchEngine/Elasticsearch/Search.pm >index ebdd1f3..2e9112a 100644 >--- a/Koha/SearchEngine/Elasticsearch/Search.pm >+++ b/Koha/SearchEngine/Elasticsearch/Search.pm >@@ -492,7 +492,7 @@ sub _convert_facets { > push @facets, $facet if exists $facet->{facets}; > } > >- @facets = sort { $a->{order} cmp $b->{order} } @facets; >+ @facets = sort { $a->{order} <=> $b->{order} } @facets; > return \@facets; > } > >-- >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 18213
:
64798
|
64841
|
64905
|
68944
|
68945
|
68946
|
72940
|
72941
|
72942
|
72943
|
73008
|
73009
|
73010
|
73011
|
75971
|
75972
|
75973
|
75974
|
85998
|
85999
|
86000
|
86001