Commit a9500662 authored by bugreport%peshkin.net's avatar bugreport%peshkin.net

Bug 234855: Show only products enterable by current user in edit-multiple

patch by kiko r=joel,justdave a=justdave
parent 0066e681
...@@ -875,7 +875,8 @@ if ($dotweak) { ...@@ -875,7 +875,8 @@ if ($dotweak) {
$vars->{'dotweak'} = 1; $vars->{'dotweak'} = 1;
$vars->{'use_keywords'} = 1 if @::legal_keywords; $vars->{'use_keywords'} = 1 if @::legal_keywords;
$vars->{'products'} = \@::legal_product; my @enterable_products = GetEnterableProducts();
$vars->{'products'} = \@enterable_products;
$vars->{'platforms'} = \@::legal_platform; $vars->{'platforms'} = \@::legal_platform;
$vars->{'priorities'} = \@::legal_priority; $vars->{'priorities'} = \@::legal_priority;
$vars->{'severities'} = \@::legal_severity; $vars->{'severities'} = \@::legal_severity;
......
...@@ -516,6 +516,17 @@ sub CanEnterProduct { ...@@ -516,6 +516,17 @@ sub CanEnterProduct {
return ($ret); return ($ret);
} }
sub GetEnterableProducts {
my @products;
# XXX rewrite into pure SQL instead of relying on legal_products?
foreach my $p (@::legal_product) {
if (CanEnterProduct($p)) {
push @products, $p;
}
}
return (@products);
}
# #
# This function returns an alphabetical list of product names to which # This function returns an alphabetical list of product names to which
# the user can enter bugs. If the $by_id parameter is true, also retrieves IDs # the user can enter bugs. If the $by_id parameter is true, also retrieves IDs
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment