improved different_target conditions

rails · Jan 8, 2013 · 3543fde · 3543fde
1 parent e5d4367
commit 3543fde
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/activerecord/lib/active_record/associations/belongs_to_association.rb b/activerecord/lib/active_record/associations/belongs_to_association.rb
@@ -50,8 +50,11 @@ def update_counters(record)
 
         # Checks whether record is different to the current target, without loading it
         def different_target?(record)
-          record.nil? && owner[reflection.foreign_key] ||
+          if record.nil? 
-          record && record.id != owner[reflection.foreign_key]
+            owner[reflection.foreign_key] 
+          else
+            record.id != owner[reflection.foreign_key]
+          end
         end
 
         def replace_keys(record)