aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record
diff options
context:
space:
mode:
authorAaron Patterson <aaron.patterson@gmail.com>2011-02-08 16:01:16 -0800
committerAaron Patterson <aaron.patterson@gmail.com>2011-02-08 16:01:16 -0800
commit1c6f4562d788cd5b63889b9597bc1765a1bd75e0 (patch)
tree675332c9f09d1f86b38f068371b64db6893d00fe /activerecord/lib/active_record
parent11061f410eca48c3a1922ade001bc54927e6b8c7 (diff)
downloadrails-1c6f4562d788cd5b63889b9597bc1765a1bd75e0.tar.gz
rails-1c6f4562d788cd5b63889b9597bc1765a1bd75e0.tar.bz2
rails-1c6f4562d788cd5b63889b9597bc1765a1bd75e0.zip
primary keys should not be cleared on cache clear, fixing oracle tests
Diffstat (limited to 'activerecord/lib/active_record')
-rw-r--r--activerecord/lib/active_record/attribute_methods/primary_key.rb1
-rw-r--r--activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb2
-rw-r--r--activerecord/lib/active_record/relation.rb9
3 files changed, 9 insertions, 3 deletions
diff --git a/activerecord/lib/active_record/attribute_methods/primary_key.rb b/activerecord/lib/active_record/attribute_methods/primary_key.rb
index 978cd7fbe3..fcdd31ddea 100644
--- a/activerecord/lib/active_record/attribute_methods/primary_key.rb
+++ b/activerecord/lib/active_record/attribute_methods/primary_key.rb
@@ -56,6 +56,7 @@ module ActiveRecord
@primary_key ||= ''
self.original_primary_key = @primary_key
value &&= value.to_s
+ connection_pool.primary_keys[table_name] = value
self.primary_key = block_given? ? instance_eval(&block) : value
end
end
diff --git a/activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb b/activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb
index b1754e61df..4297c26413 100644
--- a/activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb
+++ b/activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb
@@ -128,12 +128,10 @@ module ActiveRecord
#
# * columns
# * columns_hash
- # * primary_keys
# * tables
def clear_cache!
@columns.clear
@columns_hash.clear
- @primary_keys.clear
@tables.clear
end
diff --git a/activerecord/lib/active_record/relation.rb b/activerecord/lib/active_record/relation.rb
index 1441e9750e..852f4077f2 100644
--- a/activerecord/lib/active_record/relation.rb
+++ b/activerecord/lib/active_record/relation.rb
@@ -32,7 +32,14 @@ module ActiveRecord
def insert(values)
im = arel.compile_insert values
im.into @table
- primary_key_value = primary_key && Hash === values ? values[table[primary_key]] : nil
+
+ primary_key_value = nil
+
+ if primary_key && Hash === values
+ primary_key_value = values[values.keys.find { |k|
+ k.name == primary_key
+ }]
+ end
@klass.connection.insert(
im.to_sql,