-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix applying ON/WITH conditions to first join in Class Table Inheritance #1254
Conversation
Now we build nested joins for CTI when using ON/WITH clause.
Hello, thank you for creating this pull request. However did not open it on the "master" Please open the pull request again for the "master" branch and close Nevertheless I have opened a Jira ticket for this Pull Request to track this http://www.doctrine-project.org/jira/browse/DDC-3500 We use Jira to track the state of pull requests and the versions they got |
Hello @Ocramius. |
@vitaliyberdylo can you add a functional test involving a JTI? Ping @guilhermeblanco since you already handled this issue once before, IIRC. |
@Ocramius I didn't... it was required to implement precedence of joins (which is what this patch does for JTI) and I left aside. |
Hello @Ocramius. |
@vitaliyberdylo this will go in 2.5 for now, and may be backported later on |
I could also need a test case with a DQL query actually producing failing previously to this patch. Should be doable with some of the existing test assets. |
@Ocramius that's only the tip of the iceberg... if we go down the precedence of joins path, we should also properly address whatever got listed in http://www.doctrine-project.org/jira/browse/DDC-349 |
This PR is already applied to |
Fix applying ON/WITH conditions to first join in Class Table Inheritance
Thanks @vitaliyberdylo! |
Thank you guys! |
Now we build nested joins for CTI when using ON/WITH clause.