[ruby-core:84916] [Ruby trunk Feature#14371] New option "recursive: true" for Hash#transform_keys!

Issue #14371 has been updated by tagomoris (Satoshi TAGOMORI).


> Why prefer `each` over `transform_keys!`?
> And probably you wanted to write `elsif`.

Your points are correct. The ideal pseudo code is here:

```ruby
def transform_keys!(recursive: false, &block)
  # do original transform_keys! here
  values.each do |v|
    if v.respond_to?(:transform_keys!)
      v.transform_keys!(recursive: true, &block)
    elsif v.respond_to?(:each)
      v.each{|i| i.transform_keys!(recursive: true, &block) if i.respond_to?(:transform_keys!) }
    end
  end if recursive
end
```

----------------------------------------
Feature #14371: New option "recursive: true" for Hash#transform_keys!
https://bugs.ruby-lang.org/issues/14371#change-69626

* Author: tagomoris (Satoshi TAGOMORI)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
----------------------------------------
`Hash#transform_keys!` is available when we want to symbolize hash keys.
But in some/many cases (for example, receiver hash object is nested configuration tree loaded from any files), hash object are
nested object, which has hashes or arrays of hashes as values.
It's super useful if we can transform keys of such nested values.

My proposal is `Hash#transform_keys!(recursive: true, &:to_sym)`. Pseudo code is:

```ruby
def transform_keys!(recursive: false, &block)
  # do original transform_keys! here
  values.each do |v|
    if v.respond_to?(:each)
      v.each{|i| i.transform_keys!(recursive: true, &block) if i.respond_to?(:transform_keys!) }
    else v.respond_to?(:transform_keys!)
      v.transform_keys!(recursive: true, &block)
    end
  end if recursive
end
```

The most major usage example is: `config = MyAwesomeFormat.load(file); config.transform_keys!(recursive: true, &:to_sym)`.




-- 
https://bugs.ruby-lang.org/

Unsubscribe: <mailto:[e-mail]?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>