refactor emails service

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