From 67085930ca42fea63ab5b9616d42bb87c25889bc Mon Sep 17 00:00:00 2001 From: Garry Collum Date: Sat, 1 Aug 2009 22:49:53 -0400 Subject: [PATCH] Bug 2553: Alphabetizes branches and item-type drop-down lists in opac-topissues.pl. Signed-off-by: Galen Charlton --- opac/opac-topissues.pl | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/opac/opac-topissues.pl b/opac/opac-topissues.pl index 3371d1ddd2..6b712260c6 100755 --- a/opac/opac-topissues.pl +++ b/opac/opac-topissues.pl @@ -94,24 +94,12 @@ $template->param(do_it => 1, results_loop => \@results, ); -# load the branches ## again?? -$branches = GetBranches(); -my @branch_loop; -for my $branch_hash (sort keys %$branches ) { - my $selected=(C4::Context->userenv && ($branch_hash eq C4::Context->userenv->{branch})) if (C4::Context->preference('SearchMyLibraryFirst')); - push @branch_loop, - { - value => "$branch_hash", - branchname => $branches->{$branch_hash}->{'branchname'}, - selected => $selected - }; -} -$template->param( branchloop => \@branch_loop, "mylibraryfirst"=>C4::Context->preference("SearchMyLibraryFirst")); +$template->param( branchloop => GetBranchesLoop(C4::Context->userenv->{'branch'})); #doctype $itemtypes = GetItemTypes; my @itemtypeloop; -foreach my $thisitemtype (keys %$itemtypes) { +foreach my $thisitemtype (sort {$itemtypes->{$a}->{'description'} cmp $itemtypes->{$b}->{'description'}} keys %$itemtypes) { my %row =(value => $thisitemtype, description => $itemtypes->{$thisitemtype}->{'description'}, ); -- 2.39.5