sequel integration fix - Githubissues

pluginaweek / state_machine

Adds support for creating state machines for attributes on any Ruby class

http://www.pluginaweek.org

MIT License

3.74k stars 505 forks source link

sequel integration fix #350

Open HoneyryderChuck opened 8 years ago

HoneyryderChuck commented 8 years ago

fixing fetching default state from db column (:de…fault is db, :ruby_default is ruby)

This was generating an unnecessary warning. This is because sequel (I don't know since when, but long ago) loads column info defaults in db format and ruby format

sylvainv commented 6 years ago

Can this be merged? It is generating unnecessary warnings for me as well. Example on a state column with default value "Draft". Postgresql returns 'Draft'::text, so this is what is returned and being use to compare against "Draft". Which is not correct. :ruby_default indeed returns the proper value for comparison with state value.