From efa1199a1131ced709bf4ec1f38386104b14e0fa Mon Sep 17 00:00:00 2001 From: Frédéric Buclin Date: Tue, 21 Aug 2012 00:51:06 +0200 Subject: Bug 779747: The "Browse" link in the page header/footer doesn't sort products by classification r=dkl a=LpSolit --- enter_bug.cgi | 21 +-------------------- 1 file changed, 1 insertion(+), 20 deletions(-) (limited to 'enter_bug.cgi') diff --git a/enter_bug.cgi b/enter_bug.cgi index b571f8f44..7a8cebf43 100755 --- a/enter_bug.cgi +++ b/enter_bug.cgi @@ -25,11 +25,8 @@ use Bugzilla::Constants; use Bugzilla::Util; use Bugzilla::Error; use Bugzilla::Bug; -use Bugzilla::User; use Bugzilla::Hook; -use Bugzilla::Product; use Bugzilla::Classification; -use Bugzilla::Keyword; use Bugzilla::Token; use Bugzilla::Field; use Bugzilla::Status; @@ -67,23 +64,7 @@ if ($product_name eq '') { my @classifications; unless ($classification && $classification ne '__all') { - if (Bugzilla->params->{'useclassification'}) { - my $class; - # Get all classifications with at least one enterable product. - foreach my $product (@enterable_products) { - $class->{$product->classification_id}->{'object'} ||= - new Bugzilla::Classification($product->classification_id); - # Nice way to group products per classification, without querying - # the DB again. - push(@{$class->{$product->classification_id}->{'products'}}, $product); - } - @classifications = sort {$a->{'object'}->sortkey <=> $b->{'object'}->sortkey - || lc($a->{'object'}->name) cmp lc($b->{'object'}->name)} - (values %$class); - } - else { - @classifications = ({object => undef, products => \@enterable_products}); - } + @classifications = @{sort_products_by_classification(\@enterable_products)}; } unless ($classification) { -- cgit v1.2.3-24-g4f1b