diff options
author | Robert Speicher <robert@gitlab.com> | 2016-11-04 15:41:48 +0000 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2016-11-04 15:41:48 +0000 |
commit | 920551b5850dee2d65db3c1fbcd243ece172c508 (patch) | |
tree | 4623691372fccb7b343d1bf50959e99fe3ba4af7 | |
parent | d12e764744e3be11c7ed886fb454e09a4d1f57d6 (diff) | |
parent | 050c8dc7fc03ac356fc1f68d1d4b7396414cb7b9 (diff) | |
download | gitlab-ce-920551b5850dee2d65db3c1fbcd243ece172c508.tar.gz |
Merge branch 'rs-merge_request_params' into 'master'
Apply `*_params_ce + *_params_ee` pattern to MergeRequestsController
See merge request !7263
-rw-r--r-- | app/controllers/projects/merge_requests_controller.rb | 27 |
1 files changed, 21 insertions, 6 deletions
diff --git a/app/controllers/projects/merge_requests_controller.rb b/app/controllers/projects/merge_requests_controller.rb index 30f1cf4e5be..19afe9a3deb 100644 --- a/app/controllers/projects/merge_requests_controller.rb +++ b/app/controllers/projects/merge_requests_controller.rb @@ -589,12 +589,27 @@ class Projects::MergeRequestsController < Projects::ApplicationController end def merge_request_params - params.require(:merge_request).permit( - :title, :assignee_id, :source_project_id, :source_branch, - :target_project_id, :target_branch, :milestone_id, - :state_event, :description, :task_num, :force_remove_source_branch, - :lock_version, label_ids: [] - ) + params.require(:merge_request) + .permit(merge_request_params_ce) + end + + def merge_request_params_ce + [ + :assignee_id, + :description, + :force_remove_source_branch, + :lock_version, + :milestone_id, + :source_branch, + :source_project_id, + :state_event, + :target_branch, + :target_project_id, + :task_num, + :title, + + label_ids: [] + ] end def merge_params |