Skip to content

Commit to fix descendants and subtree methods to work properly with scopes that use joins #35

Closed
wants to merge 2 commits into from

4 participants

@raelik
raelik commented Dec 28, 2010

Because the descendants and subtree methods use raw sql as part of their conditions, using them with another scope method breaks if that scope uses joins. This commit fixes that by adding the table name to the sql.

Blane Dabney Fix descendants and subtree methods to work properly with scopes that…
… use joins by including the table name in the condition sql.
95afdd0
@kerinin
kerinin commented Dec 29, 2010

I'm having this same problem, +1 for merging into main

@swrobel
swrobel commented Jan 8, 2011

+1

@johnrees

Same problem here +1

Blane Dabney Fix ordered_by_ancestry and ordered_by_ancestry_and scopes to work pr…
…operly with scopes that use joins by including the table name in the condition sql.
5e0757a
@raelik
raelik commented Feb 14, 2011

Didn't realize I made this pull request off my master. I'm gonna close this and re-request off my branch for join fixes. Feel free to re-bump it :)

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.