diff options
author | Sean Griffin <sean@seantheprogrammer.com> | 2015-10-29 16:46:57 -0600 |
---|---|---|
committer | Sean Griffin <sean@seantheprogrammer.com> | 2015-10-29 16:53:49 -0600 |
commit | 21a386bb0726cce4f4a5d64d55fbb55d8a2b9837 (patch) | |
tree | 2938e27f69482aa6f8adaf7d9d0155bf401c20be /activerecord/lib/active_record/associations/builder | |
parent | 9f93a5efbba3e1cbf0bfa700a17ec8d1ef60d7c6 (diff) | |
download | rails-21a386bb0726cce4f4a5d64d55fbb55d8a2b9837.tar.gz rails-21a386bb0726cce4f4a5d64d55fbb55d8a2b9837.tar.bz2 rails-21a386bb0726cce4f4a5d64d55fbb55d8a2b9837.zip |
Ensure `has_and_belongs_to_many` works with `belongs_to_required_by_default`
Before this commit, if
`ActiveRecord::Base.belongs_to_required_by_default` is set to `true`,
then creating a record through `has_and_belongs_to_many` fails with the
cryptic error message `Left side must exist`. This is because
`inverse_of` isn't working properly in this case, presumably since we're
doing trickery with anonymous classes in the middle.
Rather than following this rabbit hole to try and get `inverse_of` to
work in a case that we know is not publicly supported, we can just turn
off this validation to match the behavior of 4.2 and earlier.
Diffstat (limited to 'activerecord/lib/active_record/associations/builder')
-rw-r--r-- | activerecord/lib/active_record/associations/builder/has_and_belongs_to_many.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/lib/active_record/associations/builder/has_and_belongs_to_many.rb b/activerecord/lib/active_record/associations/builder/has_and_belongs_to_many.rb index a5c9f1666e..b888148841 100644 --- a/activerecord/lib/active_record/associations/builder/has_and_belongs_to_many.rb +++ b/activerecord/lib/active_record/associations/builder/has_and_belongs_to_many.rb @@ -62,13 +62,13 @@ module ActiveRecord::Associations::Builder # :nodoc: end def self.add_left_association(name, options) - belongs_to name, options + belongs_to name, required: false, **options self.left_reflection = _reflect_on_association(name) end def self.add_right_association(name, options) rhs_name = name.to_s.singularize.to_sym - belongs_to rhs_name, options + belongs_to rhs_name, required: false, **options self.right_reflection = _reflect_on_association(rhs_name) end |