diff options
| author | Frédéric Buclin <LpSolit@gmail.com> | 2012-10-04 17:41:18 +0200 |
|---|---|---|
| committer | Frédéric Buclin <LpSolit@gmail.com> | 2012-10-04 17:41:18 +0200 |
| commit | a6f594106574a8ac3704e1a5a792ca1e79fd1222 (patch) | |
| tree | dff6116890a1ca52a58fd62f11653ae690de96a6 /Bugzilla/Group.pm | |
| parent | bcdeb0b9c577ecced7e970e53028cca5345fbfa6 (diff) | |
| download | bugzilla-a6f594106574a8ac3704e1a5a792ca1e79fd1222.tar.gz bugzilla-a6f594106574a8ac3704e1a5a792ca1e79fd1222.tar.xz | |
Bug 788098: Queries involving group substitution crash when usevisibilitygroups is enabled
r=dkl a=LpSolit
Diffstat (limited to 'Bugzilla/Group.pm')
| -rw-r--r-- | Bugzilla/Group.pm | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/Bugzilla/Group.pm b/Bugzilla/Group.pm index a5b108723..5908c86fc 100644 --- a/Bugzilla/Group.pm +++ b/Bugzilla/Group.pm @@ -173,7 +173,9 @@ sub check_members_are_visible { my $self = shift; my $user = Bugzilla->user; return if !Bugzilla->params->{'usevisibilitygroups'}; - my $is_visible = grep { $_->id == $_ } @{ $user->visible_groups_inherited }; + + my $group_id = $self->id; + my $is_visible = grep { $_ == $group_id } @{ $user->visible_groups_inherited }; if (!$is_visible) { ThrowUserError('group_not_visible', { group => $self }); } |