Doesn't need to sort, lets users of attribute_names sort them if they…

… want
rails · Dec 2, 2010 · 42c51b8 · 42c51b8
1 parent 6673d88
commit 42c51b8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/activerecord/lib/active_record/base.rb b/activerecord/lib/active_record/base.rb
@@ -1455,9 +1455,9 @@ def has_attribute?(attr_name)
         @attributes.has_key?(attr_name.to_s)
       end
 
-      # Returns an array of names for the attributes available on this object sorted alphabetically.
+      # Returns an array of names for the attributes available on this object.
       def attribute_names
-        @attributes.keys.sort
+        @attributes.keys
       end
 
       # Returns the value of the attribute identified by <tt>attr_name</tt> after it has been typecast (for example,

diff --git a/activerecord/test/cases/reflection_test.rb b/activerecord/test/cases/reflection_test.rb
@@ -25,7 +25,7 @@ def test_human_name
   def test_read_attribute_names
     assert_equal(
       %w( id title author_name author_email_address bonus_time written_on last_read content group approved replies_count parent_id parent_title type created_at updated_at ).sort,
-      @first.attribute_names
+      @first.attribute_names.sort
     )
   end