{"id":3184,"date":"2012-11-22T14:39:19","date_gmt":"2012-11-22T11:39:19","guid":{"rendered":"http:\/\/railsware.com\/blog\/?p=3184"},"modified":"2021-08-16T15:30:06","modified_gmt":"2021-08-16T12:30:06","slug":"rails-3-2-9-through-associations-issues","status":"publish","type":"post","link":"https:\/\/railsware.com\/blog\/rails-3-2-9-through-associations-issues\/","title":{"rendered":"Rails 3.2.9 through associations issues"},"content":{"rendered":"\n

Recent 3.2.9 release bugfix has brought an issue for through<\/code> associations.
inverse_of<\/code> usage can solve the problem only partially (polymorphic associations are still affected).<\/p>\n\n\n\n

UPDATE<\/span>: patch that introduced this issue has just been reverted in rails master<\/a>.<\/p>\n\n\n\n

Few days ago Rails 3.2.9 was released.
Among the bugfixes a fix for through<\/code> associations<\/a> is present, and since 3.2.9 join model instances are built even though the object is not saved yet.<\/p>\n\n\n\n

To get the point take a look at these simple models with User<\/code> and Team<\/code> joined with each other through<\/code> UserMembership<\/code>:<\/p>\n\n\n\n

class User < ActiveRecord::Base\n  has_many :user_memberships\n  has_many :teams, through: :user_memberships\nend\n\nclass Team < ActiveRecord::Base\n  has_many :user_memberships\n  has_many :users, through: :user_memberships\nend\n\nclass UserMembership < ActiveRecord::Base\n  belongs_to :team\n  belongs_to :user\n  validates :team, :user, presence: true\nend\n<\/pre>\n\n\n\n
Before Rails 3.2.9 this code does not build any UserMembership<\/code> instances:<\/p>\n\n\n\n
team = Team.new(users: [User.first])\nteam.valid?           #=> true\nteam.user_memberships #=> []\n<\/pre>\n\n\n\n
But since Rails 3.2.9 UserMembership<\/code> instance is built:<\/p>\n\n\n\n
team = Team.new(users: [User.first])\nteam.valid?                      #=> false\nteam.user_memberships            #=> [#]\nteam.user_memberships.first.team #=> nil\n<\/pre>\n\n\n\n
Oh, have you noticed that team<\/code> is invalid now? Aahh, now when it get’s complicated.<\/p>\n\n\n\n
Built instance of a join UserMembership<\/code> model
has no team<\/code> set and that’s why it’s invalid (it has validate :team, :user, presence: true<\/code>).
As soon as join model instance is invalid team<\/code> is invalid as well.<\/p>\n\n\n\n
To fix this validation issues in this particular case the one should use inverse_of<\/code> for the associations like this:<\/p>\n\n\n\n
class User < ActiveRecord::Base\n  has_many :user_memberships, :inverse_of => :user\n  has_many :teams, through: :user_memberships\nend\n\nclass Team < ActiveRecord::Base\n  has_many :user_memberships, :inverse_of => :team\n  has_many :users, through: :user_memberships\nend\n\nteam = Team.new(users: [User.first])\nteam.valid?                        #=> true\nteam.user_memberships.first.valid? #=> true\nteam.user_memberships.first.team   #=> #\n<\/pre>\n\n\n\n
inverse_of<\/code> takes care of associations objects in-memory synchronization, so now UserMembership<\/code> instance has a team<\/code> and it’s valid.
Unfortunately polymorphic associations are still affected – follow this github discussion<\/a> for more info.<\/p>\n\n\n\n
That said, if you’ve just upgraded to 3.2.9 and your tests are failing review the associations and fix them whenever it’s possible by using inverse_of<\/code>.<\/p>\n","protected":false},"excerpt":{"rendered":"
Recent 3.2.9 release bugfix has brought an issue for through associations.inverse_of usage can solve the problem only partially (polymorphic associations are still affected). UPDATE: patch that introduced this issue has just been reverted in rails master. Few days ago Rails 3.2.9 was released.Among the bugfixes a fix for through associations is present, and since 3.2.9…<\/p>\n","protected":false},"author":34,"featured_media":3196,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"_acf_changed":false,"inline_featured_image":false,"footnotes":""},"categories":[3],"tags":[],"coauthors":["Innokenty Mihailov"],"class_list":["post-3184","post","type-post","status-publish","format-standard","has-post-thumbnail","hentry","category-development"],"acf":[],"aioseo_notices":[],"aioseo_head":"\n\t\t\n\t\n\t\n\t\n\t\n\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t\n\t\t