summaryrefslogtreecommitdiffstats
path: root/globals.pl
diff options
context:
space:
mode:
Diffstat (limited to 'globals.pl')
-rw-r--r--globals.pl8
1 files changed, 4 insertions, 4 deletions
diff --git a/globals.pl b/globals.pl
index 07eb573f5..d1680959e 100644
--- a/globals.pl
+++ b/globals.pl
@@ -482,7 +482,7 @@ sub CanEditProductId {
my $query = "SELECT group_id FROM group_control_map " .
"WHERE product_id = $productid " .
"AND canedit != 0 ";
- if (defined Bugzilla->user && %{Bugzilla->user->groups}) {
+ if (%{Bugzilla->user->groups}) {
$query .= "AND group_id NOT IN(" .
join(',', values(%{Bugzilla->user->groups})) . ") ";
}
@@ -504,7 +504,7 @@ sub CanEnterProduct {
"LEFT JOIN group_control_map " .
"ON group_control_map.product_id = products.id " .
"AND group_control_map.entry != 0 ";
- if (defined Bugzilla->user && %{Bugzilla->user->groups}) {
+ if (%{Bugzilla->user->groups}) {
$query .= "AND group_id NOT IN(" .
join(',', values(%{Bugzilla->user->groups})) . ") ";
}
@@ -547,7 +547,7 @@ sub GetSelectableProducts {
$query .= "AND group_control_map.membercontrol = " .
CONTROLMAPMANDATORY . " ";
}
- if (defined Bugzilla->user && %{Bugzilla->user->groups}) {
+ if (%{Bugzilla->user->groups}) {
$query .= "AND group_id NOT IN(" .
join(',', values(%{Bugzilla->user->groups})) . ") ";
}
@@ -1192,7 +1192,7 @@ sub UserInGroup {
die "UserInGroup no longer takes a second parameter.";
}
- return defined Bugzilla->user && defined Bugzilla->user->groups->{$_[0]};
+ return defined Bugzilla->user->groups->{$_[0]};
}
sub UserCanBlessGroup {