Use the new interface for NotificationRecipientService

22476cad · Lin Jen-Shin · 5ca1dd2a · 22476cad
Commit 22476cad authored Aug 05, 2017 by Lin Jen-Shin
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

app/services/notification_service.rb app/services/notification_service.rb +2 -2

No files found.
--- a/app/services/notification_service.rb
+++ b/app/services/notification_service.rb
@@ -414,7 +414,7 @@ class NotificationService
  end

  def approve_mr_email(merge_request, project, current_user)
-    recipients = NotificationRecipientService.new(project).build_recipients(merge_request, current_user, action: 'approve')
+    recipients = NotificationRecipientService.build_recipients(merge_request, current_user, action: 'approve')

    recipients.each do |recipient|
      mailer.approved_merge_request_email(recipient.id, merge_request.id, current_user.id).deliver_later
@@ -422,7 +422,7 @@ class NotificationService
  end

  def unapprove_mr_email(merge_request, project, current_user)
-    recipients = NotificationRecipientService.new(project).build_recipients(merge_request, current_user, action: 'unapprove')
+    recipients = NotificationRecipientService.build_recipients(merge_request, current_user, action: 'unapprove')

    recipients.each do |recipient|
      mailer.unapproved_merge_request_email(recipient.id, merge_request.id, current_user.id).deliver_later