Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Revert "Makes AR use AMo to_key implementation"

This reverts commit ccd4364.
  • Loading branch information...
commit 10177d3a38fb441065ee5d664e880cdb403b1da7 1 parent f224c66
@spastorino spastorino authored
View
6 activerecord/lib/active_record/attribute_methods/primary_key.rb
@@ -3,6 +3,12 @@ module AttributeMethods
module PrimaryKey
extend ActiveSupport::Concern
+ # Returns this record's primary key value wrapped in an Array
+ # or nil if the record is a new_record?
+ def to_key
+ new_record? ? nil : [ id ]
+ end
+
module ClassMethods
# Defines the primary key field -- can be overridden in subclasses. Overwriting will negate any effect of the
# primary_key_prefix_type setting, though.
View
2  activerecord/test/cases/primary_keys_test.rb
@@ -26,7 +26,7 @@ def test_to_key_with_customized_primary_key
def test_to_key_with_primary_key_after_destroy
topic = Topic.find(1)
topic.destroy
- assert_equal nil, topic.to_key
+ assert_equal [1], topic.to_key
end
def test_integer_key
Please sign in to comment.
Something went wrong with that request. Please try again.