Bugzilla – Attachment 191748 Details for
Bug 39001
Language list is displayed in wrong order (getLanguages sorts it by language code; places that display language list to the end-user in UI should re-sort that by localized display name order)
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 39001: Sort language search limits by description
d09c0b6.patch (text/plain), 1.66 KB, created by
Fridolin Somers
on 2026-01-21 10:13:06 UTC
(
hide
)
Description:
Bug 39001: Sort language search limits by description
Filename:
MIME Type:
Creator:
Fridolin Somers
Created:
2026-01-21 10:13:06 UTC
Size:
1.66 KB
patch
obsolete
>From d09c0b6abd0064a4a5c4d3e66757d15c7399eb44 Mon Sep 17 00:00:00 2001 >From: Fridolin Somers <fridolin.somers@biblibre.com> >Date: Wed, 21 Jan 2026 11:02:22 +0100 >Subject: [PATCH] Bug 39001: Sort language search limits by description > >--- > catalogue/search.pl | 4 +++- > opac/opac-search.pl | 4 +++- > 2 files changed, 6 insertions(+), 2 deletions(-) > >diff --git a/catalogue/search.pl b/catalogue/search.pl >index 762d952ae7..3cc87f431b 100755 >--- a/catalogue/search.pl >+++ b/catalogue/search.pl >@@ -333,7 +333,9 @@ if ( $template_type eq 'advsearch' ) { > > # load the language limits (for search) > my $languages_limit_loop = getLanguages( $lang, 1 ); >- $template->param( search_languages_loop => $languages_limit_loop, ); >+ my @sorted_languages_limit_loop = >+ sort { $a->{language_description} cmp $b->{language_description} } @$languages_limit_loop; >+ $template->param( search_languages_loop => \@sorted_languages_limit_loop, ); > } > > # Expanded search options in advanced search: >diff --git a/opac/opac-search.pl b/opac/opac-search.pl >index db9dc776c3..19abe17a0f 100755 >--- a/opac/opac-search.pl >+++ b/opac/opac-search.pl >@@ -229,7 +229,9 @@ if ( $template_type && $template_type eq 'advsearch' ) { > > # load the language limits (for search) > my $languages_limit_loop = getLanguages( $lang, 1 ); >- $template->param( search_languages_loop => $languages_limit_loop, ); >+ my @sorted_languages_limit_loop = >+ sort { $a->{language_description} cmp $b->{language_description} } @$languages_limit_loop; >+ $template->param( search_languages_loop => \@sorted_languages_limit_loop, ); > } > > # load the Type stuff >-- >2.43.0 >
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 39001
: 191748