diff options
author | Brett Walker <brett@digitalmoksha.com> | 2017-09-12 17:36:59 +0200 |
---|---|---|
committer | Brett Walker <brett@digitalmoksha.com> | 2017-09-23 15:24:53 +0200 |
commit | 30e3a49e23f8ce0c2a07b13addc45951d7fc6719 (patch) | |
tree | c08428fd3331f8c5f638bd1e38916fe44ba2ed78 /app/services/emails | |
parent | a9b31786971d83c193a1430df7c5c4550ba5aa6b (diff) | |
download | gitlab-ce-30e3a49e23f8ce0c2a07b13addc45951d7fc6719.tar.gz |
optimized the email services
Diffstat (limited to 'app/services/emails')
-rw-r--r-- | app/services/emails/base_service.rb | 2 | ||||
-rw-r--r-- | app/services/emails/confirm_service.rb | 4 | ||||
-rw-r--r-- | app/services/emails/destroy_service.rb | 4 |
3 files changed, 5 insertions, 5 deletions
diff --git a/app/services/emails/base_service.rb b/app/services/emails/base_service.rb index ace49889097..227c87602fc 100644 --- a/app/services/emails/base_service.rb +++ b/app/services/emails/base_service.rb @@ -1,6 +1,6 @@ module Emails class BaseService - def initialize(user, opts) + def initialize(user, opts = {}) @user = user @email = opts[:email] end diff --git a/app/services/emails/confirm_service.rb b/app/services/emails/confirm_service.rb index 45845ccecc5..e764f18ddd0 100644 --- a/app/services/emails/confirm_service.rb +++ b/app/services/emails/confirm_service.rb @@ -1,7 +1,7 @@ module Emails class ConfirmService < ::Emails::BaseService - def execute - Email.find_by_email!(@email).resend_confirmation_instructions + def execute(email_record) + email_record.resend_confirmation_instructions end end end diff --git a/app/services/emails/destroy_service.rb b/app/services/emails/destroy_service.rb index d586b9dfe0c..d29d7e69bde 100644 --- a/app/services/emails/destroy_service.rb +++ b/app/services/emails/destroy_service.rb @@ -1,7 +1,7 @@ module Emails class DestroyService < ::Emails::BaseService - def execute - Email.find_by_email!(@email).destroy && update_secondary_emails! + def execute(email_record) + email_record.destroy && update_secondary_emails! end private |