Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Makes AR use AMo to_key implementation

  • Loading branch information...
commit 36a84a4f15f29b41c7cac2f8de410055006a8a8d 1 parent 48c7ad1
@spastorino spastorino authored fxn committed
View
6 activerecord/lib/active_record/attribute_methods/primary_key.rb
@@ -3,12 +3,6 @@ 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 [1], topic.to_key
+ assert_equal nil, topic.to_key
end
def test_integer_key
Please sign in to comment.
Something went wrong with that request. Please try again.