diff options
| -rw-r--r-- | app/services/emails/create_service.rb | 2 | ||||
| -rw-r--r-- | app/services/emails/destroy_service.rb | 2 | ||||
| -rw-r--r-- | lib/api/users.rb | 4 |
3 files changed, 4 insertions, 4 deletions
diff --git a/app/services/emails/create_service.rb b/app/services/emails/create_service.rb index 5d5655256fa..95e226ec710 100644 --- a/app/services/emails/create_service.rb +++ b/app/services/emails/create_service.rb @@ -1,5 +1,5 @@ module Emails - class CreateService < BaseService + class CreateService < ::Emails::BaseService def execute(skip_authorization: false) raise Gitlab::Access::AccessDeniedError unless skip_authorization || can_manage_emails? diff --git a/app/services/emails/destroy_service.rb b/app/services/emails/destroy_service.rb index c5fbc32033b..1275d31efb0 100644 --- a/app/services/emails/destroy_service.rb +++ b/app/services/emails/destroy_service.rb @@ -1,5 +1,5 @@ module Emails - class DestroyService < BaseService + class DestroyService < ::Emails::BaseService def execute(skip_authorization: false) raise Gitlab::Access::AccessDeniedError unless skip_authorization || can_manage_emails? diff --git a/lib/api/users.rb b/lib/api/users.rb index 236bae4a299..940f8b64026 100644 --- a/lib/api/users.rb +++ b/lib/api/users.rb @@ -276,7 +276,7 @@ module API email = user.emails.find_by(id: params[:email_id]) not_found!('Email') unless email - Emails::DestroyService.new(current_user, self, email: email.email).execute + Emails::DestroyService.new(current_user, user, email: email.email).execute ::Users::UpdateService.new(current_user, user).execute do |user| user.update_secondary_emails! @@ -510,7 +510,7 @@ module API email = current_user.emails.find_by(id: params[:email_id]) not_found!('Email') unless email - Emails::DestroyService.new(current_user, self, email: email.email).execute + Emails::DestroyService.new(current_user, current_user, email: email.email).execute ::Users::UpdateService.new(current_user, current_user).execute do |user| user.update_secondary_emails! |