diff options
author | James Lopez <james@jameslopez.es> | 2017-06-14 12:35:58 +0300 |
---|---|---|
committer | James Lopez <james@jameslopez.es> | 2017-06-23 12:41:41 +0300 |
commit | 59c3968c40a1146f5370ae3b634aac2d4a045733 (patch) | |
tree | ad607fb897beb405ce0651c983a9eba0ddd9ef68 /app/services | |
parent | cf6286313d2e5296e1009c3bcae7da50d7e438a4 (diff) |
use update service on ldap call and updated specs and service
Diffstat (limited to 'app/services')
-rw-r--r-- | app/services/users/update_service.rb | 24 |
1 files changed, 20 insertions, 4 deletions
diff --git a/app/services/users/update_service.rb b/app/services/users/update_service.rb index ac20473b6eb..866eb070913 100644 --- a/app/services/users/update_service.rb +++ b/app/services/users/update_service.rb @@ -7,16 +7,32 @@ module Users @params = params.dup end - def execute(skip_authorization: false) - raise Gitlab::Access::AccessDeniedError unless skip_authorization || can_update_user? + def execute(skip_authorization: false, &block) + assign_attributes(skip_authorization, &block) - if @user.update_attributes(params) + if @user.save success else - error('Project could not be updated') + error('User could not be updated') end end + def execute!(skip_authorization: false, &block) + assign_attributes(skip_authorization, &block) + + @user.save! + end + + private + + def assign_attributes(skip_authorization, &block) + raise Gitlab::Access::AccessDeniedError unless skip_authorization || can_update_user? + + yield(@user) if block_given? + + @user.assign_attributes(params) if params.any? + end + def can_update_user? current_user == @user || current_user&.admin? end |