diff options
| author | Byron Jones <bjones@mozilla.com> | 2013-05-29 08:35:00 +0200 |
|---|---|---|
| committer | Byron Jones <bjones@mozilla.com> | 2013-05-29 08:35:00 +0200 |
| commit | 405866228897e5c22db42ca8ea2418a8fca00a5e (patch) | |
| tree | ae6859c1f72c123706367010430c50b8a66335bd /extensions/Splinter/Extension.pm | |
| parent | fccfb94637fcf01beadf77ee58620097b7070224 (diff) | |
| download | bugzilla-405866228897e5c22db42ca8ea2418a8fca00a5e.tar.gz bugzilla-405866228897e5c22db42ca8ea2418a8fca00a5e.tar.xz | |
Bug 760989: Splinter should not require login
Diffstat (limited to 'extensions/Splinter/Extension.pm')
| -rw-r--r-- | extensions/Splinter/Extension.pm | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/extensions/Splinter/Extension.pm b/extensions/Splinter/Extension.pm index eef9e89af..c383297c4 100644 --- a/extensions/Splinter/Extension.pm +++ b/extensions/Splinter/Extension.pm @@ -33,8 +33,7 @@ sub page_before_template { my ($vars, $page) = @$args{qw(vars page_id)}; if ($page eq 'splinter.html') { - # Login is required for performing a review - my $user = Bugzilla->login(LOGIN_REQUIRED); + my $user = Bugzilla->user; # We can either provide just a bug id to see a list # of prior reviews by the user, or just an attachment @@ -54,7 +53,8 @@ sub page_before_template { # Check to see if the user can see the bug this attachment is connected to. Bugzilla::Bug->check($attachment->bug_id); - if ($attachment->isprivate && $user->id != $attachment->attacher->id + if ($attachment->isprivate + && $user->id != $attachment->attacher->id && !$user->is_insider) { ThrowUserError('auth_failure', {action => 'access', |