diff options
author | James Lopez <james@jameslopez.es> | 2017-06-22 08:55:07 +0200 |
---|---|---|
committer | James Lopez <james@jameslopez.es> | 2017-06-23 11:41:42 +0200 |
commit | 785cbb79e255c8369ca5eb916207304f39d188ad (patch) | |
tree | abd792d12be1f35d868db591793311eb32d92370 /app/services/emails | |
parent | 0c8e7f49d1ef32ed5ea1bdd7e26dd5e169bad359 (diff) | |
download | gitlab-ce-785cbb79e255c8369ca5eb916207304f39d188ad.tar.gz |
refactor emails service
Diffstat (limited to 'app/services/emails')
-rw-r--r-- | app/services/emails/destroy_service.rb | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/app/services/emails/destroy_service.rb b/app/services/emails/destroy_service.rb index 1275d31efb0..8150918986c 100644 --- a/app/services/emails/destroy_service.rb +++ b/app/services/emails/destroy_service.rb @@ -3,7 +3,17 @@ module Emails def execute(skip_authorization: false) raise Gitlab::Access::AccessDeniedError unless skip_authorization || can_manage_emails? - Email.find_by_email(@email).destroy + Email.find_by_email(@email).destroy && update_secondary_emails! + end + + private + + def update_secondary_emails! + result = ::Users::UpdateService.new(@current_user, @current_user).execute do |user| + user.update_secondary_emails! + end + + result[:status] == 'success' end end end |