Bugzilla – Attachment 40314 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]
[SIGNED-OFF] Bug 14371 - Facets should be sorted by label (displayed) not title (link value)
SIGNED-OFF-Bug-14371---Facets-should-be-sorted-by-.patch (text/plain), 1.53 KB, created by
Bernardo Gonzalez Kriegel
on 2015-06-18 15:06:37 UTC
(
hide
)
Description:
[SIGNED-OFF] Bug 14371 - Facets should be sorted by label (displayed) not title (link value)
Filename:
MIME Type:
Creator:
Bernardo Gonzalez Kriegel
Created:
2015-06-18 15:06:37 UTC
Size:
1.53 KB
patch
obsolete
>From 46b5e1ab0b07d4cd9aee9c459d85296019eb79a0 Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@quecheelibrary.org> >Date: Tue, 9 Jun 2015 21:51:44 -0400 >Subject: [PATCH] [SIGNED-OFF] 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 > >Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> >Work as described, no errors >--- > 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.7.9.5
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