diff options
| author | Robert Speicher <rspeicher@gmail.com> | 2012-08-10 18:51:07 -0400 |
|---|---|---|
| committer | Robert Speicher <rspeicher@gmail.com> | 2012-09-16 10:06:29 -0400 |
| commit | 8cfb197dfcad3aa0c11f7afd34e4c7bfef953d1d (patch) | |
| tree | e2dfa88831e0e806c810f5e7979595a91fbd25a5 /app/controllers/issues_controller.rb | |
| parent | a0893b4d58a6162743b325efe0f93b1b957e8a0a (diff) | |
| download | gitlab-ce-8cfb197dfcad3aa0c11f7afd34e4c7bfef953d1d.tar.gz | |
Remove redundant access check for Issues#destroy
We were already calling `authorize_admin_issue!` in a before filter with
the same permission checks, so this deleted check wasn't actually doing
anything.
Diffstat (limited to 'app/controllers/issues_controller.rb')
| -rw-r--r-- | app/controllers/issues_controller.rb | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/app/controllers/issues_controller.rb b/app/controllers/issues_controller.rb index 9be37ca1da4..1d78a6d9b80 100644 --- a/app/controllers/issues_controller.rb +++ b/app/controllers/issues_controller.rb @@ -87,8 +87,6 @@ class IssuesController < ApplicationController end def destroy - return access_denied! unless can?(current_user, :admin_issue, @issue) - @issue.destroy respond_to do |format| |