diff options
| author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2016-03-29 13:52:42 +0200 |
|---|---|---|
| committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2016-03-30 10:44:20 +0200 |
| commit | 71e7b398431506c8bac2e8e6014b0f3891a41f95 (patch) | |
| tree | 31d30ecd7d71de5614cc79e5213789cb3c5f7539 /app/models/member.rb | |
| parent | 630c86a7a36cee36ed6b9c93644a6cb51e2b3b23 (diff) | |
| download | gitlab-ce-71e7b398431506c8bac2e8e6014b0f3891a41f95.tar.gz | |
Refactor creating notification setting with defaults
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Diffstat (limited to 'app/models/member.rb')
| -rw-r--r-- | app/models/member.rb | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/app/models/member.rb b/app/models/member.rb index cbcc54c0250..747d0f16d8d 100644 --- a/app/models/member.rb +++ b/app/models/member.rb @@ -163,12 +163,7 @@ class Member < ActiveRecord::Base end def create_notification_setting - notification_setting = user.notification_settings.find_or_initialize_by(source: source) - - unless notification_setting.persisted? - notification_setting.set_defaults - notification_setting.save - end + user.notification_setting.find_or_create_for(source) end def notification_setting |