Commit 5b2b80f0 authored by Maxim Rydkin's avatar Maxim Rydkin Committed by Rémy Coutable

refactor Ci::CreatePipelineService#allowed_to_create?

parent 7913a163
...@@ -100,16 +100,16 @@ module Ci ...@@ -100,16 +100,16 @@ module Ci
end end
def allowed_to_create? def allowed_to_create?
access = Gitlab::UserAccess.new(current_user, project: project) return unless can?(current_user, :create_pipeline, project)
can?(current_user, :create_pipeline, project) && access = Gitlab::UserAccess.new(current_user, project: project)
if branch? if branch?
access.can_update_branch?(ref) access.can_update_branch?(ref)
elsif tag? elsif tag?
access.can_create_tag?(ref) access.can_create_tag?(ref)
else else
true # Allow it for now and we'll reject when we check ref existence true # Allow it for now and we'll reject when we check ref existence
end end
end end
def update_merge_requests_head_pipeline def update_merge_requests_head_pipeline
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment