diff options
| author | Dmitriy Zaporozhets <dzaporozhets@gitlab.com> | 2015-01-21 07:08:31 +0000 |
|---|---|---|
| committer | Dmitriy Zaporozhets <dzaporozhets@gitlab.com> | 2015-01-21 07:08:31 +0000 |
| commit | 1050f5230eec21cf47d5af262a1b3e62c07fec5d (patch) | |
| tree | 7a32b98b19476c6ca633f9cb907f3f9c6e1f6bb8 /lib/api | |
| parent | 148740cc6769b0faf7ee564552143ccd0a18768b (diff) | |
| parent | ab7a79bf3bb47fd1c9d82da0bb29a3cdf0246cdc (diff) | |
| download | gitlab-ce-1050f5230eec21cf47d5af262a1b3e62c07fec5d.tar.gz | |
Merge branch 'protected_branches' into 'master'
Developer can push to protected branch if allowed
#1876
See merge request !1418
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 |