diff options
author | Valery Sizov <valery@gitlab.com> | 2015-01-20 15:23:37 -0800 |
---|---|---|
committer | Valery Sizov <valery@gitlab.com> | 2015-01-20 17:34:23 -0800 |
commit | ab7a79bf3bb47fd1c9d82da0bb29a3cdf0246cdc (patch) | |
tree | 7a32b98b19476c6ca633f9cb907f3f9c6e1f6bb8 /lib/api | |
parent | 148740cc6769b0faf7ee564552143ccd0a18768b (diff) | |
download | gitlab-ce-ab7a79bf3bb47fd1c9d82da0bb29a3cdf0246cdc.tar.gz |
developer can push to protected branches
Diffstat (limited to 'lib/api')
-rw-r--r-- | lib/api/merge_requests.rb | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/lib/api/merge_requests.rb b/lib/api/merge_requests.rb index 81038d05f12..2a5b10c6f52 100644 --- a/lib/api/merge_requests.rb +++ b/lib/api/merge_requests.rb @@ -167,13 +167,9 @@ module API put ":id/merge_request/:merge_request_id/merge" do merge_request = user_project.merge_requests.find(params[:merge_request_id]) - action = if user_project.protected_branch?(merge_request.target_branch) - :push_code_to_protected_branches - else - :push_code - end + allowed = ::Gitlab::GitAccess.can_push_to_branch?(current_user, user_project, merge_request.target_branch) - if can?(current_user, action, user_project) + if allowed if merge_request.unchecked? merge_request.check_if_can_be_merged end |