Implement #to_hash #25

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

bekkou68 commented Dec 1, 2011

Before, #to_hash returned an object of Class which extends Settingslogic,
not that of pure Hash. So we couldn't use methods of Hash.
Now #to_hash becomes to return a pure Hash object and we can do that.

Takumi Tsunokake Implement #to_hash
Before, #to_hash returned an object of Class which extends Settingslogic,
not that of *pure* Hash. So we couldn't use methods of Hash.
Now #to_hash becomes to return a pure Hash object and we can do that.
933562b

This is the problem what I am facing
I serialize Hash into DB
Since it is Settings but not Hash, it returns with much extra information (from app.yml) which I don't need
Please pull this quick!

Edit:
For those who want to use this method before pulled
just add the method "to_hash" from the commit into your own "Settings" model

bekkou68 commented Mar 2, 2012

dear @PikachuEXE,

thanks for your comment with a nice example :)

bekkou68 commented Jun 8, 2012

hi, how's going?

I think you might just want to fork it and continue= =

bekkou68 commented Jun 9, 2012

yah :/

@bekkou68 #44 may be a better implementation
But you spec is better :)

thanks for your comment and creating wiki page ;)
i hope our pull requests are merged into the official code base soon.

Owner

Functionality was pulled in. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment