Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix bad or_groups SQL for determining if a user can see a product. r=LpSolit, a=justdave.

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@265246 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
bzrmirror%bugzilla.org 2014-02-17 17:00:48 +00:00
parent 8124a0f52d
commit 076726d426
2 changed files with 19 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
mozilla/webtools/bugzilla/.bzrrev
View File

@ -1 +1 @@
8921
8922

38
mozilla/webtools/bugzilla/Bugzilla/User.pm
View File

@ -1057,26 +1057,24 @@ sub get_selectable_products {
my $class_restricted = Bugzilla->params->{'useclassification'} && $class_id;
if (!defined $self->{selectable_products}) {
my $query =
Bugzilla->params->{'or_groups'}
? "SELECT id
FROM products
WHERE id NOT IN (
SELECT product_id
FROM group_control_map
WHERE group_control_map.membercontrol = " . CONTROLMAPMANDATORY . "
AND group_id NOT IN (" . $self->groups_as_string . ")
)
ORDER BY name"
: "SELECT id
FROM products
LEFT JOIN group_control_map
ON group_control_map.product_id = products.id
AND group_control_map.membercontrol = " . CONTROLMAPMANDATORY . "
AND group_id NOT IN(" . $self->groups_as_string . ")
WHERE group_id IS NULL
ORDER BY name";
my $query = "SELECT id
FROM products
LEFT JOIN group_control_map
ON group_control_map.product_id = products.id
AND group_control_map.membercontrol = " . CONTROLMAPMANDATORY;
if (Bugzilla->params->{'or_groups'}) {
# Either the user is in at least one of the MANDATORY groups, or
# there are no such groups for the product.
$query .= " WHERE group_id IN (" . $self->groups_as_string . ")
OR group_id IS NULL";
}
else {
# There must be no MANDATORY groups that the user is not in.
$query .= " AND group_id NOT IN (" . $self->groups_as_string . ")
WHERE group_id IS NULL";
}
my $prod_ids = Bugzilla->dbh->selectcol_arrayref($query);
$self->{selectable_products} = Bugzilla::Product->new_from_list($prod_ids);
}