mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Make ActiveSupport::OrderedHash
extractable when using Array#extract_options!
`ActiveSupport::OrderedHash` is actually a subclass of the hash, so it does make sense that it should be extractable from the array list.
This commit is contained in:
parent
36f1612c34
commit
9984266c04
3 changed files with 12 additions and 0 deletions
|
@ -1,5 +1,7 @@
|
|||
*Rails 3.2.0 (unreleased)*
|
||||
|
||||
* ActiveSupport::OrderedHash is now marked as extractable when using Array#extract_options! [Prem Sichanugrist]
|
||||
|
||||
* Added Array#prepend as an alias for Array#unshift and Array#append as an alias for Array#<< [DHH]
|
||||
|
||||
* The definition of blank string for Ruby 1.9 has been extended to Unicode whitespace.
|
||||
|
|
|
@ -47,6 +47,11 @@ module ActiveSupport
|
|||
self
|
||||
end
|
||||
|
||||
# Returns true to make sure that this hash is extractable via <tt>Array#extract_options!</tt>
|
||||
def extractable_options?
|
||||
true
|
||||
end
|
||||
|
||||
# Hash is ordered in Ruby 1.9!
|
||||
if RUBY_VERSION < '1.9'
|
||||
|
||||
|
|
|
@ -329,4 +329,9 @@ class OrderedHashTest < Test::Unit::TestCase
|
|||
assert_equal expected, @ordered_hash.invert
|
||||
assert_equal @values.zip(@keys), @ordered_hash.invert.to_a
|
||||
end
|
||||
|
||||
def test_extractable
|
||||
@ordered_hash[:rails] = "snowman"
|
||||
assert_equal @ordered_hash, [1, 2, @ordered_hash].extract_options!
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue