-
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
DDC-2780: IS [NOT] NULL conditions with JOINs #3528
Comments
Comment created by maryo: OK. players.id IS NULL (but it's uglier) |
Comment created by Soullivaneuh:
But I don't understand why this will be not fixed. I agree, it's uglier. |
Comment created by maryo: I am checking the activity and it looks It was me who changed it to "Won't fix". If it's true then it was an accident. Sorry if I'm wrong. |
Comment created by pejot: Hi, also experiencing this error. Worked fine with 2.4.7, now trying to migrate to 2.5.0 and getting this error. The problem is, even when I do select the variable to SELECT, still getting the error. |
Comment created by mrthehud: I'm seeing this too - but I actually think the "players.id IS NULL" is right - otherwise what you're really asking for is something like "COUNT(r.players) == 0" - which might cause problems when you combine the query with pagination (for instance, using the KNP Paginator in Symfony). |
Comment created by benjamin: Still no updates on this bug, two years after? I'm surprised that a potential bug introduced by a commit is left without any comment from the core team for so long. |
Any news on this? |
Any news here? |
There is probably a reason the exception was added. IMHO it would be helpful, if someone proved that #5680 solves it. Otherwise the maintainers have to do it, which means they can do it only when they have the time for it. |
Jira issue originally created by user maryo:
leftJoin('r.players', 'players', 'WITH', 'players = :player') ->where('players IS NULL'); It was working 3 month ago. But then, this commit broke it. https://github.com/doctrine/doctrine2/commit/d9c1782a4f6d46f66e9deb2c375830f9192d4482 Or in other words if this is not a bug. How to test if the collection does not contain let's say a user with ID 5 (AND XX OR YY) with DQL/queryBuilder API? IN SQL I would just simply LEFT JOIN it with ON condition and then i would test it for existence using IS NULL in WHERE condition. How to do this using DQL when the pasted example ends with an error?The text was updated successfully, but these errors were encountered: