summaryrefslogtreecommitdiff
path: root/app/services/emails
diff options
context:
space:
mode:
authorBrett Walker <brett@digitalmoksha.com>2017-09-12 17:36:59 +0200
committerBrett Walker <brett@digitalmoksha.com>2017-09-23 15:24:53 +0200
commit30e3a49e23f8ce0c2a07b13addc45951d7fc6719 (patch)
treec08428fd3331f8c5f638bd1e38916fe44ba2ed78 /app/services/emails
parenta9b31786971d83c193a1430df7c5c4550ba5aa6b (diff)
downloadgitlab-ce-30e3a49e23f8ce0c2a07b13addc45951d7fc6719.tar.gz
optimized the email services
Diffstat (limited to 'app/services/emails')
-rw-r--r--app/services/emails/base_service.rb2
-rw-r--r--app/services/emails/confirm_service.rb4
-rw-r--r--app/services/emails/destroy_service.rb4
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