Actions
Bug #19541
closed
UserMailNotification records are not set up properly when a mail notification is added.
Description
When assigning one-to-many associations we should use after(:create) hook to add those associations properly.
Updated by The Foreman Bot over 7 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4531 added
Updated by
Updated by Shimon Shtein over 7 years ago
- Subject changed from Fix user factory's :with_mail_notification trait to UserMailNotification records are not set up properly when a mail notification is added.
When using new in-memory objects, the association objects are not set up correctly without :inverse_of option.
to reproduce:
User.new(:login => 'zzz', :mail_notifications => [MailNotification.new]).user_mail_notifications.first.user
This statement will return nil, but should return a reference to the newly created user.
This happens due to the fact that inverse_of is not set.
After setting proper inverse_of, Active Record will point UserMailNotification#user property to the newly created user in this example.
Updated by
Updated by Shimon Shtein over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 7b08519908a379a0d55a0fd81f0215bdec619a33.
Updated by Marek Hulán over 7 years ago
- Translation missing: en.field_release set to 240
Actions