From 0e3cc7da193e9a94bbd797a0e3139a20f66d84d2 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 4 Jan 2016 10:02:06 +0000 Subject: [PATCH] Bug 15354: Sort itemtypes by translated description everywhere on circ rules admin page It's better to cmp on lc. --- admin/smart-rules.pl | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/admin/smart-rules.pl b/admin/smart-rules.pl index 810c79f..bf4f9e0 100755 --- a/admin/smart-rules.pl +++ b/admin/smart-rules.pl @@ -437,6 +437,7 @@ while (my $data=$sth->fetchrow_hashref){ $sth->finish; my @row_loop; my @itemtypes = @{ GetItemTypes( style => 'array' ) }; +@itemtypes = sort { lc $a->{translated_description} cmp lc $b->{translated_description} } @itemtypes; my $sth2 = $dbh->prepare(" SELECT issuingrules.*, @@ -540,7 +541,7 @@ my @branch_item_rules = (); while (my $row = $sth_branch_item->fetchrow_hashref) { push @branch_item_rules, $row; } -my @sorted_branch_item_rules = sort { $a->{translated_description} cmp $b->{translated_description} } @branch_item_rules; +my @sorted_branch_item_rules = sort { lc $a->{translated_description} cmp lc $b->{translated_description} } @branch_item_rules; # note undef holdallowed so that template can deal with them foreach my $entry (@sorted_branch_item_rules) { @@ -619,6 +620,6 @@ sub by_itemtype { } elsif ($b->{'default_translated_description'}) { return -1; } else { - return $a->{'translated_description'} cmp $b->{'translated_description'}; + return lc $a->{'translated_description'} cmp lc $b->{'translated_description'}; } } -- 2.1.0