Skip to content
New issue

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

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

Already on GitHub? # to your account

Workaround for squeel, fixes #178 #179

Closed
wants to merge 1 commit into from

Conversation

godfat
Copy link

@godfat godfat commented Dec 22, 2014

Workaround squeel bug:
activerecord-hackery/squeel#355

@godfat
Copy link
Author

godfat commented Dec 22, 2014

Unfortunately this does not seem to fix all cases. It's giving wrong result with:

Impression::ActiveRecord_Associations_CollectionProxy

At least it's not giving a syntax error though.

@godfat
Copy link
Author

godfat commented Dec 22, 2014

Eventually I need to write something like this:
godfat@0a6cdeb
You probably don't want to merge this though, since it's really ugly...
However this pull request would still at least stop syntax error.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants