Merge remote-tracking branch 'upstream/pull/2050'

[rails.git] / app / controllers / issue_comments_controller.rb

diff --git a/app/controllers/issue_comments_controller.rb b/app/controllers/issue_comments_controller.rb
index 42ae850ba24a039594ee3a9cb9cc6bccea4c5270..0e4a7079e0e4c5e27608bf344767d529a4349e1f 100644 (file)
--- a/app/controllers/issue_comments_controller.rb
+++ b/app/controllers/issue_comments_controller.rb
@@ -2,15 +2,16 @@ class IssueCommentsController < ApplicationController
   layout "site"
 
   before_action :authorize_web
-  before_action :require_user
-  before_action :check_permission
+  before_action :set_locale
+
+  authorize_resource
 
   def create
     @issue = Issue.find(params[:issue_id])
     comment = @issue.comments.build(issue_comment_params)
     comment.user = current_user
     comment.save!
-    notice = t("issues.comment.comment_created")
+    notice = t(".comment_created")
     reassign_issue(@issue) if params[:reassign]
     redirect_to @issue, :notice => notice
   end
@@ -21,10 +22,12 @@ class IssueCommentsController < ApplicationController
     params.require(:issue_comment).permit(:body)
   end
 
-  def check_permission
-    unless current_user.administrator? || current_user.moderator?
+  def deny_access(_exception)
+    if current_user
       flash[:error] = t("application.require_moderator_or_admin.not_a_moderator_or_admin")
       redirect_to root_path
+    else
+      super
     end
   end