Use notificationService for MR reassigned

1 files changed, 3 insertions, 12 deletions

diff --git a/app/observers/merge_request_observer.rb b/app/observers/merge_request_observer.rb
index 355f7848d15..64585570362 100644
--- a/app/observers/merge_request_observer.rb
+++ b/app/observers/merge_request_observer.rb
@@ -6,29 +6,20 @@ class MergeRequestObserver < ActiveRecord::Observer
   end
 
   def after_close(merge_request, transition)
-    send_reassigned_email(merge_request) if merge_request.is_being_reassigned?
-
     Note.create_status_change_note(merge_request, current_user, merge_request.state)
   end
 
   def after_reopen(merge_request, transition)
-    send_reassigned_email(merge_request) if merge_request.is_being_reassigned?
-
     Note.create_status_change_note(merge_request, current_user, merge_request.state)
   end
 
   def after_update(merge_request)
-    send_reassigned_email(merge_request) if merge_request.is_being_reassigned?
+    notification.reassigned_merge_request(merge_request) if merge_request.is_being_reassigned?
   end
 
   protected
 
-  def send_reassigned_email(merge_request)
-    recipients_ids = merge_request.assignee_id_was, merge_request.assignee_id
-    recipients_ids.delete current_user.id
-
-    recipients_ids.each do |recipient_id|
-      Notify.delay.reassigned_merge_request_email(recipient_id, merge_request.id, merge_request.assignee_id_was)
-    end
+  def notification
+    NotificationService.new
   end
 end