Bugzilla – Attachment 72943 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), 925 bytes, created by
Nick Clemens
on 2018-03-15 12:55:01 UTC
(
hide
)
Description:
Bug 18213: (follow-up) Correctly sort facets
Filename:
MIME Type:
Creator:
Nick Clemens
Created:
2018-03-15 12:55:01 UTC
Size:
925 bytes
patch
obsolete
>From 79071e18f944c2508f99bfadc758769186196cef 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 >--- > 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 8cd3619..c2c00a1 100644 >--- a/Koha/SearchEngine/Elasticsearch/Search.pm >+++ b/Koha/SearchEngine/Elasticsearch/Search.pm >@@ -496,7 +496,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.1.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