From f2b3deb85cab1389d6b895b2e45b33054a96a6ba Mon Sep 17 00:00:00 2001 From: Marcel Molina Date: Sun, 14 May 2006 18:17:30 +0000 Subject: Don't destroy a HashWithIndifferentAccess if symbolize_keys! or stringify_keys! is called on it. Closes #5076. [Marcel Molina Jr., guy.naor@famundo.com] git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4339 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- activesupport/CHANGELOG | 2 ++ .../lib/active_support/core_ext/hash/indifferent_access.rb | 3 +++ activesupport/test/core_ext/hash_ext_test.rb | 11 +++++++++++ 3 files changed, 16 insertions(+) diff --git a/activesupport/CHANGELOG b/activesupport/CHANGELOG index 52e8c5abe6..a651cff958 100644 --- a/activesupport/CHANGELOG +++ b/activesupport/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Don't destroy a HashWithIndifferentAccess if symbolize_keys! or stringify_keys! is called on it. Closes #5076. [Marcel Molina Jr., guy.naor@famundo.com] + * Document Module::delegate. #5002 [pergesu@gmail.com] * Replace alias method chaining with Module#alias_method_chain. [Marcel Molina Jr.] diff --git a/activesupport/lib/active_support/core_ext/hash/indifferent_access.rb b/activesupport/lib/active_support/core_ext/hash/indifferent_access.rb index 472e3ab7d1..1f645fabc9 100644 --- a/activesupport/lib/active_support/core_ext/hash/indifferent_access.rb +++ b/activesupport/lib/active_support/core_ext/hash/indifferent_access.rb @@ -56,6 +56,9 @@ class HashWithIndifferentAccess < Hash def delete(key) super(convert_key(key)) end + + def stringify_keys!; self end + def symbolize_keys!; self end protected def convert_key(key) diff --git a/activesupport/test/core_ext/hash_ext_test.rb b/activesupport/test/core_ext/hash_ext_test.rb index 3fc3b86254..a12299d32c 100644 --- a/activesupport/test/core_ext/hash_ext_test.rb +++ b/activesupport/test/core_ext/hash_ext_test.rb @@ -145,6 +145,17 @@ class HashExtTest < Test::Unit::TestCase assert_equal hash.delete('a'), nil end + def test_stringify_and_symbolize_keys_on_indifferent_preserves_hash + h = HashWithIndifferentAccess.new + h[:first] = 1 + h.stringify_keys! + assert_equal 1, h['first'] + h = HashWithIndifferentAccess.new + h['first'] = 1 + h.symbolize_keys! + assert_equal 1, h[:first] + end + def test_assert_valid_keys assert_nothing_raised do { :failure => "stuff", :funny => "business" }.assert_valid_keys([ :failure, :funny ]) -- cgit v1.2.3