Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement "symbolize_keys" method #33

Open
vovayartsev opened this issue Jan 30, 2012 · 4 comments
Open

Implement "symbolize_keys" method #33

vovayartsev opened this issue Jan 30, 2012 · 4 comments

Comments

@vovayartsev
Copy link

In v2.0.8 gem, 'symbolize_keys' method neither works nor raises any exceptions, which can lead to unexpected results, e.g.:

Settings.smtp_settings                  # => {"address"=>"smtp.sendgrid.net", "port"=>"587", .....
Settings.smtp_settings.symbolize_keys   # => {"address"=>"smtp.sendgrid.net", "port"=>"587", .....

It would be better if symbolize_keys method worked just like in Hash class
@alex-kovshovik
Copy link

+1

goosetav pushed a commit to goosetav/settingslogic that referenced this issue Jun 22, 2012
symbolize_keys method. fix for binarylogic#33
f890392
@goosetav
Copy link
Contributor

I had the exact same use case as @vovayartsev

@atipugin
Copy link

atipugin commented Jul 4, 2012

+1 for this!

@ddonia
Copy link

ddonia commented Oct 5, 2012

+1

@goosetav goosetav mentioned this issue Oct 8, 2012
Merged
binarylogic added a commit that referenced this issue Dec 30, 2012
@binarylogic
Merge pull request #38 from goosetav/master
c510aff 
symbolize_keys for issue #33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@goosetav @vovayartsev @atipugin @alex-kovshovik @ddonia