OSDN Git Service

Remove redundant access check for Issues#destroy
authorRobert Speicher <rspeicher@gmail.com>
Fri, 10 Aug 2012 22:51:07 +0000 (18:51 -0400)
committerRobert Speicher <rspeicher@gmail.com>
Sun, 16 Sep 2012 14:06:29 +0000 (10:06 -0400)
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.

app/controllers/issues_controller.rb

index 9be37ca..1d78a6d 100644 (file)
@@ -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|