X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/a20e28809f5ca1e3f3579faeaf118fff3ecc4994..f3a9d2e9d61f3f596d559ac75d54c08bc60fc629:/app/controllers/issue_comments_controller.rb?ds=inline diff --git a/app/controllers/issue_comments_controller.rb b/app/controllers/issue_comments_controller.rb index 448b2a5d6..5bf4d0237 100644 --- a/app/controllers/issue_comments_controller.rb +++ b/app/controllers/issue_comments_controller.rb @@ -2,17 +2,32 @@ class IssueCommentsController < ApplicationController layout "site" before_action :authorize_web - before_action :require_user - before_action :check_permission + before_action :set_locale + before_action :check_database_readable + + authorize_resource + + before_action :check_database_writable, :only => [:create] 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") - reassign_issue(@issue) if params[:reassign] - redirect_to @issue, :notice => notice + + if params[:reassign] + reassign_issue(@issue) + flash[:notice] = t ".issue_reassigned" + + if current_user.role? @issue.assigned_role + redirect_to @issue + else + redirect_to issues_path(:status => "open") + end + else + flash[:notice] = t(".comment_created") + redirect_to @issue + end end private @@ -21,13 +36,6 @@ class IssueCommentsController < ApplicationController params.require(:issue_comment).permit(:body) end - def check_permission - unless current_user.administrator? || current_user.moderator? - flash[:error] = t("application.require_admin.not_an_admin") - redirect_to root_path - end - end - # This sort of assumes there are only two roles def reassign_issue(issue) role = (Issue::ASSIGNED_ROLES - [issue.assigned_role]).first