aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord
diff options
context:
space:
mode:
authorEmilio Tagua <miloops@gmail.com>2010-10-20 19:27:37 -0300
committerEmilio Tagua <miloops@gmail.com>2010-11-19 19:08:58 -0300
commit7d563f9278a748df001697ee66af73c7761fed29 (patch)
treeab2fd7dce4f208a3d5cb304adb342b448e4ca739 /activerecord
parent6cd1224f2eae69a44c820918657408b7650c1288 (diff)
downloadrails-7d563f9278a748df001697ee66af73c7761fed29.tar.gz
rails-7d563f9278a748df001697ee66af73c7761fed29.tar.bz2
rails-7d563f9278a748df001697ee66af73c7761fed29.zip
Refactor associations cache removal from IM. (ht: Aaron Patterson)
Diffstat (limited to 'activerecord')
-rw-r--r--activerecord/lib/active_record/associations.rb8
1 files changed, 4 insertions, 4 deletions
diff --git a/activerecord/lib/active_record/associations.rb b/activerecord/lib/active_record/associations.rb
index 12b0601e3c..fd9f000147 100644
--- a/activerecord/lib/active_record/associations.rb
+++ b/activerecord/lib/active_record/associations.rb
@@ -118,10 +118,10 @@ module ActiveRecord
def clear_association_cache #:nodoc:
self.class.reflect_on_all_associations.to_a.each do |assoc|
if IdentityMap.enabled? && instance_variable_defined?("@#{assoc.name}")
- targets = [*instance_variable_get("@#{assoc.name}")]
- targets.map! { |t| t.respond_to?(:target) ? t.target : t }
- targets.compact!
- targets.each { |r| IdentityMap.remove r }
+ Array(instance_variable_get("@#{assoc.name}")).each do |t|
+ next unless t.respond_to?(:target)
+ IdentityMap.remove t.target unless t.target.nil?
+ end
end
instance_variable_set "@#{assoc.name}", nil
end if self.persisted?