Changeset 7217

Timestamp:

07/24/07 14:17:09 (1 year ago)

Author:

david

Message:

Added Array#extract_options! to encapsulate the pattern of getting an options hash out of a variable number of parameters (closes #8759) [norbert]

Files:

• trunk/activesupport/CHANGELOG (modified) (1 diff)
• trunk/activesupport/lib/active_support/core_ext/array.rb (modified) (1 diff)
• trunk/activesupport/lib/active_support/core_ext/array/extract_options.rb (added)
• trunk/activesupport/test/core_ext/array_ext_test.rb (modified) (1 diff)

trunk/activesupport/CHANGELOG

@@ -1 +1 @@
-*SVN*
+
+* Added Array#extract_options! to encapsulate the pattern of getting an options hash out of a variable number of parameters #8759 [norbert].
-
-* Let alias_attribute work with attributes with initial capital letters (legacy columns etc).  Closes #8596 [mpalmer]

trunk/activesupport/lib/active_support/core_ext/array.rb

@@ -1 +1 @@
-require File.dirname(__FILE__) + '/array/conversions'
-require File.dirname(__FILE__) + '/array/grouping'
+require File.dirname(__FILE__) + '/array/extract_options'
-
-class Array #:nodoc:
-  include ActiveSupport::CoreExtensions::Array::Conversions
-  include ActiveSupport::CoreExtensions::Array::Grouping
+  include ActiveSupport::CoreExtensions::Array::ExtractOptions
-end

trunk/activesupport/test/core_ext/array_ext_test.rb

@@ -192 +192 @@
-  end
-end
+
+class ArrayExtractOptionsTests < Test::Unit::TestCase
+  def test_extract_options
+    assert_equal({}, [].extract_options!)
+    assert_equal({}, [1].extract_options!)
+    assert_equal({:a=>:b}, [{:a=>:b}].extract_options!)
+    assert_equal({:a=>:b}, [1, {:a=>:b}].extract_options!)
+  end
+end