diff options
| author | Richard Macklin <richard.github@nrm.com> | 2017-02-03 21:01:45 -0800 |
|---|---|---|
| committer | Richard Macklin <richard.github@nrm.com> | 2017-02-03 21:01:45 -0800 |
| commit | 7cd260b10727cef0621ecef429ec89e00873b1b7 (patch) | |
| tree | fc3d7974bd39fdc90f16c7e2f8d738c43fc15b11 | |
| parent | 0e2c96e709161b35daeae2dad3aefcf9c85a75aa (diff) | |
| download | gitlab-ce-7cd260b10727cef0621ecef429ec89e00873b1b7.tar.gz | |
Refactor NotificationService#pipeline_finished to use skip_current_user
instead of passing nil for current_user
| -rw-r--r-- | app/services/notification_service.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/app/services/notification_service.rb b/app/services/notification_service.rb index 5a7d5ef8747..e5283720913 100644 --- a/app/services/notification_service.rb +++ b/app/services/notification_service.rb @@ -327,8 +327,9 @@ class NotificationService recipients ||= build_recipients( pipeline, pipeline.project, - nil, # The acting user, who won't be added to recipients - action: pipeline.status).map(&:notification_email) + pipeline.user, + action: pipeline.status, + skip_current_user: false).map(&:notification_email) if recipients.any? mailer.public_send(email_template, pipeline, recipients).deliver_later |