aboutsummaryrefslogtreecommitdiffstats
path: root/activesupport/lib
diff options
context:
space:
mode:
authoryuuji.yaginuma <yuuji.yaginuma@gmail.com>2017-09-27 13:18:34 +0900
committeryuuji.yaginuma <yuuji.yaginuma@gmail.com>2017-09-27 14:02:12 +0900
commitde9e3238a008f8f20c5beedefb8787b6e51359d0 (patch)
treef46bd3643758534647907f5529e9e3d89aeee77e /activesupport/lib
parent48ef5e0ad500bdba706f10b65456f3b03b556212 (diff)
downloadrails-de9e3238a008f8f20c5beedefb8787b6e51359d0.tar.gz
rails-de9e3238a008f8f20c5beedefb8787b6e51359d0.tar.bz2
rails-de9e3238a008f8f20c5beedefb8787b6e51359d0.zip
Ensure `HashWithIndifferentAccess#transform_keys` to return `HashWithIndifferentAccess`
Currently, `#transform_values`, `#select` and `#reject` return instance of `HashWithIndifferentAccess`. But `#transform_keys` returns instance of Hash. This behavior is a bit confusing. I think that `HashWithIndifferentAccess#transform_keys` should also return instance of `HashWithIndifferentAccess` as well as other methods.
Diffstat (limited to 'activesupport/lib')
-rw-r--r--activesupport/lib/active_support/hash_with_indifferent_access.rb5
1 files changed, 5 insertions, 0 deletions
diff --git a/activesupport/lib/active_support/hash_with_indifferent_access.rb b/activesupport/lib/active_support/hash_with_indifferent_access.rb
index 12291af443..fcc13feb8c 100644
--- a/activesupport/lib/active_support/hash_with_indifferent_access.rb
+++ b/activesupport/lib/active_support/hash_with_indifferent_access.rb
@@ -306,6 +306,11 @@ module ActiveSupport
dup.tap { |hash| hash.transform_values!(*args, &block) }
end
+ def transform_keys(*args, &block)
+ return to_enum(:transform_keys) unless block_given?
+ dup.tap { |hash| hash.transform_keys!(*args, &block) }
+ end
+
def compact
dup.tap(&:compact!)
end