diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2014-06-26 14:30:07 +0300 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2014-06-26 14:30:07 +0300 |
commit | 1dab15940db4c77ac23f49ece7eee2847d4614aa (patch) | |
tree | c786ef9b9605e487aff180c2baab94b36ebe5835 /app/services/projects/update_service.rb | |
parent | 024e0577c6b6ded30ee09536082bd24405bda1e5 (diff) | |
download | gitlab-ce-1dab15940db4c77ac23f49ece7eee2847d4614aa.tar.gz |
Remove protected_atrributes gem and start moving to strong params
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Diffstat (limited to 'app/services/projects/update_service.rb')
-rw-r--r-- | app/services/projects/update_service.rb | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/app/services/projects/update_service.rb b/app/services/projects/update_service.rb index 551a3653cad..d21bba69b51 100644 --- a/app/services/projects/update_service.rb +++ b/app/services/projects/update_service.rb @@ -1,19 +1,19 @@ module Projects class UpdateService < BaseService - def execute(role = :default) - params[:project].delete(:namespace_id) + def execute + params.delete(:namespace_id) # check that user is allowed to set specified visibility_level - unless can?(current_user, :change_visibility_level, project) && Gitlab::VisibilityLevel.allowed_for?(current_user, params[:project][:visibility_level]) - params[:project].delete(:visibility_level) + unless can?(current_user, :change_visibility_level, project) && Gitlab::VisibilityLevel.allowed_for?(current_user, params[:visibility_level]) + params.delete(:visibility_level) end - new_branch = params[:project].delete(:default_branch) + new_branch = params.delete(:default_branch) if project.repository.exists? && new_branch && new_branch != project.default_branch project.change_head(new_branch) end - if project.update_attributes(params[:project], as: role) + if project.update_attributes(params) if project.previous_changes.include?('namespace_id') project.send_move_instructions end |