Remove `try` from NotificationService#build_recipients

After refactoring pipeline_finished to avoid passing `nil` for current_user, we shouldn't need to use `try` here anymore.

Remove `try` from NotificationService#build_recipients
9493791d · Richard Macklin · 7cd260b1 · 9493791d
Commit 9493791d authored Feb 03, 2017 by Richard Macklin
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

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

No files found.
--- a/app/services/notification_service.rb
+++ b/app/services/notification_service.rb
@@ -628,7 +628,7 @@ class NotificationService
    recipients = reject_unsubscribed_users(recipients, target)
    recipients = reject_users_without_access(recipients, target)

-    recipients.delete(current_user) if skip_current_user && !current_user.try(:notified_of_own_activity?)
+    recipients.delete(current_user) if skip_current_user && !current_user.notified_of_own_activity?

    recipients.uniq
  end