diff options
author | Byron Jones <bjones@mozilla.com> | 2013-12-18 06:55:28 +0100 |
---|---|---|
committer | Byron Jones <bjones@mozilla.com> | 2013-12-18 06:55:28 +0100 |
commit | ed1e4230e934cd59962096b2c4621acf20d7c0f7 (patch) | |
tree | f8da97e35efd0a890fdc4bba278422abb3c52c6f /extensions/Review | |
parent | 0947dc7814a092dd39a50b37de2a608b0fe9e1c6 (diff) | |
download | bugzilla-ed1e4230e934cd59962096b2c4621acf20d7c0f7.tar.gz bugzilla-ed1e4230e934cd59962096b2c4621acf20d7c0f7.tar.xz |
Bug 951016: X-Bugzilla-Mentors missing from email
Diffstat (limited to 'extensions/Review')
-rw-r--r-- | extensions/Review/Extension.pm | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/extensions/Review/Extension.pm b/extensions/Review/Extension.pm index f15bdc9ef..c2951d29b 100644 --- a/extensions/Review/Extension.pm +++ b/extensions/Review/Extension.pm @@ -80,6 +80,7 @@ sub _bug_mentors { if (!exists $self->{mentors}) { my @mentors; my $whiteboard = $self->status_whiteboard; + my $logout = 0; while ($whiteboard =~ /\[mentor=([^\]]+)\]/g) { my $mentor_string = $1; my $user; @@ -89,6 +90,13 @@ sub _bug_mentors { } else { # otherwise assume it's a : prefixed nick. only works if a # single user matches. + + # we need to be logged in to do user searching + if (!Bugzilla->user->id) { + Bugzilla->set_user(Bugzilla::User->check({ name => 'nobody@mozilla.org' })); + $logout = 1; + } + foreach my $query ("*:$mentor_string*", "*$mentor_string*") { my $matches = Bugzilla::User::match($query, 2); if ($matches && scalar(@$matches) == 1) { @@ -99,6 +107,7 @@ sub _bug_mentors { } push @mentors, $user if $user; } + Bugzilla->logout_request() if $logout; $self->{mentors} = \@mentors; } return $self->{mentors}; |