Bugzilla – Attachment 40054 Details for
Bug 14371
Facets should be sorted by label (displayed) not title (link value)
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 14371 - Facets should be sorted by label (displayed) not title (link value)
Bug-14371---Facets-should-be-sorted-by-label-displ.patch (text/plain), 1.42 KB, created by
Nick Clemens (kidclamp)
on 2015-06-10 02:00:16 UTC
(
hide
)
Description:
Bug 14371 - Facets should be sorted by label (displayed) not title (link value)
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2015-06-10 02:00:16 UTC
Size:
1.42 KB
patch
obsolete
>From 3d9a68eb2846794e62a7fb711b1bc6d000d3cb4b Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@quecheelibrary.org> >Date: Tue, 9 Jun 2015 21:51:44 -0400 >Subject: [PATCH] Bug 14371 - Facets should be sorted by label (displayed) not > title (link value) > >This patch changes one small line in search.pl to sort facets by: >facet_label_value >instead of >facet_title_value > >To test: >1 - Perform a search with results in two branches e.g. Centerville (code CPL) and Fairfield (code FPL) >2 - Notice that branch facets appear correctly sorted >3 - Rename the branches Centervile->Zebra and Fairfeild->Aardvark (but don't change codes) >4 - Repeat original search >5 - Note that branch facets are no longer correctly sorted >6 - Apply patch >7 - Repeat search >8 - Facets should be correctly sorted >9 - Ensure there are no unintended consequences/regressions >--- > catalogue/search.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/catalogue/search.pl b/catalogue/search.pl >index 4d165ac..006a1d4 100755 >--- a/catalogue/search.pl >+++ b/catalogue/search.pl >@@ -536,7 +536,7 @@ eval { > # This sorts the facets into alphabetical order > if ($facets) { > foreach my $f (@$facets) { >- $f->{facets} = [ sort { uc($a->{facet_title_value}) cmp uc($b->{facet_title_value}) } @{ $f->{facets} } ]; >+ $f->{facets} = [ sort { uc($a->{facet_label_value}) cmp uc($b->{facet_label_value}) } @{ $f->{facets} } ]; > } > } > if ($@ || $error) { >-- >1.9.1
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 14371
:
40054
|
40314
|
40324
|
40381
|
40382