Use instance variable for CensorRule in #destroy

Easier to check against the assigned variable, but the spec still fails
because an internal error is raised if the rule is not associated with
a User or InfoRequest

1 files changed, 9 insertions, 6 deletions

diff --git a/app/controllers/admin_censor_rule_controller.rb b/app/controllers/admin_censor_rule_controller.rb
index 18c40410c..68ca57510 100644
--- a/app/controllers/admin_censor_rule_controller.rb
+++ b/app/controllers/admin_censor_rule_controller.rb
@@ -95,19 +95,22 @@ class AdminCensorRuleController < AdminController
     end
 
     def destroy
-        censor_rule = CensorRule.find(params[:censor_rule_id])
-        info_request = censor_rule.info_request
-        user = censor_rule.user
+        @censor_rule = CensorRule.find(params[:censor_rule_id])
+        info_request = @censor_rule.info_request
+        user = @censor_rule.user
 
-        censor_rule.destroy
-        if !info_request.nil?
+        @censor_rule.destroy
+
+        unless info_request.nil?
             expire_for_request(info_request)
         end
-        if !user.nil?
+
+        unless user.nil?
             expire_requests_for_user(user)
         end
 
         flash[:notice] = "CensorRule was successfully destroyed."
+
         if !info_request.nil?
             redirect_to admin_request_show_url(info_request)
         elsif !user.nil?