Bugzilla – Attachment 15761 Details for
Bug 3549
Framework sorting in addbook
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 3549 -[Revised]- Framework sorting in addbook
Bug-3549--Revised--Framework-sorting-in-addbook.patch (text/plain), 1.36 KB, created by
Bernardo Gonzalez Kriegel
on 2013-02-28 11:11:19 UTC
(
hide
)
Description:
Bug 3549 -[Revised]- Framework sorting in addbook
Filename:
MIME Type:
Creator:
Bernardo Gonzalez Kriegel
Created:
2013-02-28 11:11:19 UTC
Size:
1.36 KB
patch
obsolete
>From 71b14063e0b95c347f0b58a2e7d4b16b4bf613e3 Mon Sep 17 00:00:00 2001 >From: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> >Date: Thu, 21 Feb 2013 14:37:07 -0300 >Subject: [PATCH] Bug 3549 -[Revised]- Framework sorting in addbook > >Reviewing Bug 9649, this bug comes up. > >This patch correctly sort frameworks by description. > >To test: >1) Go to Cataloguing home, button 'New record' shows >unordered list of frameworks >2) Apply the patch >3) refresh, the list must be correctly sorted. > >Revised to sort using uppercase. > >Signed-off-by: Owen Leonard <oleonard@myacpl.org> >--- > cataloguing/addbooks.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/cataloguing/addbooks.pl b/cataloguing/addbooks.pl >index 7bca2f3..c8a8071 100755 >--- a/cataloguing/addbooks.pl >+++ b/cataloguing/addbooks.pl >@@ -57,7 +57,7 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( > # get framework list > my $frameworks = getframeworks; > my @frameworkcodeloop; >-foreach my $thisframeworkcode ( sort {$frameworks->{$a} cmp $frameworks->{$b}}keys %{$frameworks} ) { >+foreach my $thisframeworkcode ( sort { uc($frameworks->{$a}->{'frameworktext'}) cmp uc($frameworks->{$b}->{'frameworktext'}) } keys %{$frameworks} ) { > push @frameworkcodeloop, { > value => $thisframeworkcode, > frameworktext => $frameworks->{$thisframeworkcode}->{'frameworktext'}, >-- >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 3549
:
1407
|
15592
|
15594
|
15761
|
15762