diff options
| author | Richard Macklin <richard.github@nrm.com> | 2017-02-03 21:03:26 -0800 |
|---|---|---|
| committer | Richard Macklin <richard.github@nrm.com> | 2017-02-03 21:03:26 -0800 |
| commit | 9493791d1212f484217e74757550353a5ef07dcf (patch) | |
| tree | 2512300f631905b5ac3501ae1bb02122f2c405f7 | |
| parent | 7cd260b10727cef0621ecef429ec89e00873b1b7 (diff) | |
| download | gitlab-ce-9493791d1212f484217e74757550353a5ef07dcf.tar.gz | |
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.
| -rw-r--r-- | app/services/notification_service.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/services/notification_service.rb b/app/services/notification_service.rb index e5283720913..3734e3c4253 100644 --- 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 |