diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2018-05-26 11:56:32 +0900 |
---|---|---|
committer | Ryuta Kamizono <kamipo@gmail.com> | 2018-05-26 11:56:32 +0900 |
commit | e34c2050e996ffe7ae5feecb005117cd7df4ba74 (patch) | |
tree | e1a3cfbef28001b5554472fbed71d2241c38197e /activerecord/lib | |
parent | 6edf354b656ec556c3573402019504a519531953 (diff) | |
download | rails-e34c2050e996ffe7ae5feecb005117cd7df4ba74.tar.gz rails-e34c2050e996ffe7ae5feecb005117cd7df4ba74.tar.bz2 rails-e34c2050e996ffe7ae5feecb005117cd7df4ba74.zip |
Fix `different_target?` to respect custom primary key counter
Diffstat (limited to 'activerecord/lib')
-rw-r--r-- | activerecord/lib/active_record/associations/belongs_to_association.rb | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/activerecord/lib/active_record/associations/belongs_to_association.rb b/activerecord/lib/active_record/associations/belongs_to_association.rb index 775559e224..121592733c 100644 --- a/activerecord/lib/active_record/associations/belongs_to_association.rb +++ b/activerecord/lib/active_record/associations/belongs_to_association.rb @@ -86,12 +86,15 @@ module ActiveRecord # Checks whether record is different to the current target, without loading it def different_target?(record) - record.id != owner._read_attribute(reflection.foreign_key) + record._read_attribute(primary_key(record)) != owner._read_attribute(reflection.foreign_key) end def replace_keys(record) - owner[reflection.foreign_key] = record ? - record._read_attribute(reflection.association_primary_key(record.class)) : nil + owner[reflection.foreign_key] = record ? record._read_attribute(primary_key(record)) : nil + end + + def primary_key(record) + reflection.association_primary_key(record.class) end def foreign_key_present? |