meineerde/redmine
1
0
Fork 0
mirror of https://github.com/meineerde/redmine.git synced 2025-12-20 15:31:12 +00:00
Code Issues Releases Wiki Activity

cleanup: rubocop: fix Layout/EmptyLineAfterGuardClause in app/controllers/issues_controller.rb

Browse Source 
git-svn-id: http://svn.redmine.org/redmine/trunk@19178 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Toshi MARUYAMA 2019-11-22 16:50:04 +00:00
parent e82fcb3fc5
commit 4a9d5faaf1
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/controllers/issues_controller.rb
View File

@ -137,6 +137,7 @@ class IssuesController < ApplicationController
unless User.current.allowed_to?(:add_issues, @issue.project, :global => true) unless User.current.allowed_to?(:add_issues, @issue.project, :global => true)
raise ::Unauthorized raise ::Unauthorized
end end
call_hook(:controller_issues_new_before_save, { :params => params, :issue => @issue }) call_hook(:controller_issues_new_before_save, { :params => params, :issue => @issue })
@issue.save_attachments(params[:attachments] || (params[:issue] && params[:issue][:uploads])) @issue.save_attachments(params[:attachments] || (params[:issue] && params[:issue][:uploads]))
if @issue.save if @issue.save
@ -181,6 +182,7 @@ class IssuesController < ApplicationController
def update def update
return unless update_issue_from_params return unless update_issue_from_params
@issue.save_attachments(params[:attachments] || @issue.save_attachments(params[:attachments] ||
(params[:issue] && params[:issue][:uploads])) (params[:issue] && params[:issue][:uploads]))
saved = false saved = false
@ -219,6 +221,7 @@ class IssuesController < ApplicationController
def issue_tab def issue_tab
return render_error :status => 422 unless request.xhr? return render_error :status => 422 unless request.xhr?
tab = params[:name] tab = params[:name]
case tab case tab
@ -332,6 +335,7 @@ class IssuesController < ApplicationController
unless User.current.allowed_to?(:copy_issues, @projects) unless User.current.allowed_to?(:copy_issues, @projects)
raise ::Unauthorized raise ::Unauthorized
end end
target_projects = @projects target_projects = @projects
if attributes['project_id'].present? if attributes['project_id'].present?
target_projects = Project.where(:id => attributes['project_id']).to_a target_projects = Project.where(:id => attributes['project_id']).to_a
@ -339,6 +343,7 @@ class IssuesController < ApplicationController
unless User.current.allowed_to?(:add_issues, target_projects) unless User.current.allowed_to?(:add_issues, target_projects)
raise ::Unauthorized raise ::Unauthorized
end end
unless User.current.allowed_to?(:add_issue_watchers, @projects) unless User.current.allowed_to?(:add_issue_watchers, @projects)
copy_watchers = false copy_watchers = false
end end
@ -539,6 +544,7 @@ class IssuesController < ApplicationController
unless User.current.allowed_to?(:copy_issues, @copy_from.project) unless User.current.allowed_to?(:copy_issues, @copy_from.project)
raise ::Unauthorized raise ::Unauthorized
end end
@link_copy = link_copy?(params[:link_copy]) || request.get? @link_copy = link_copy?(params[:link_copy]) || request.get?
@copy_attachments = params[:copy_attachments].present? || request.get? @copy_attachments = params[:copy_attachments].present? || request.get?
@copy_subtasks = params[:copy_subtasks].present? || request.get? @copy_subtasks = params[:copy_subtasks].present? || request.get?