Fixing the Arel::Nodes::SqlLiteral Psych dump issue #169

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants
@mdesantis

I don't know if it is an acceptable solution; anyway, I defined custom encode_with and init_with methods for Arel::Nodes::SqlLiteral which restores the Psych dump of SqlLiteral instances (issue #149).

Custom encode_with and init_with (fixes #149)
It declares encode_with and init_with for Arel::Nodes::SqlLiteral, which restores the dump / load with Psych.
@prathamesh-sonpatki

This comment has been minimized.

Show comment Hide comment
@prathamesh-sonpatki

prathamesh-sonpatki Jul 29, 2013

Member

Got this from @codetriage. @prognommers any updates on this issue?

Got this from @codetriage. @prognommers any updates on this issue?

@mdesantis

This comment has been minimized.

Show comment Hide comment
@mdesantis

mdesantis Jul 29, 2013

I'm still using the monkeypatch of the issue #149 and I've never got problems (it is inside the code of a couple of web apps in production)

I'm still using the monkeypatch of the issue #149 and I've never got problems (it is inside the code of a couple of web apps in production)

@prathamesh-sonpatki

This comment has been minimized.

Show comment Hide comment
@prathamesh-sonpatki

prathamesh-sonpatki Jul 30, 2013

Member
@tamird

This comment has been minimized.

Show comment Hide comment
@tamird

tamird Sep 16, 2014

obsolete: was fixed in fcb00d3. good to close

tamird commented Sep 16, 2014

obsolete: was fixed in fcb00d3. good to close

@matthewd matthewd closed this Sep 16, 2014

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