-
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-2229: Undefined method #2927
Comments
Comment created by @FabioBatSilva: Hi Karma Could you provide the failing DQL ? Cheers |
Comment created by drak: It's generating SQL from a Doctrine Collection in a proxy class. Here is a link to the example: zikula/core#674 (comment) The SQL being generated is
The ? should be the value 2, but for some reason it's not being added. I've checked the files in the stack-trace below and the right values are entering into the process, just the wrong SQL seems to be generated. Exception Trace |
Comment created by @FabioBatSilva: Hi Drak, Could you try to write a failing test case please ? Cheers |
Comment created by @asm89: ping! :) |
Comment created by hussdl:
However,
If you still need a failing test case, I'll write one when I get home from work. |
Comment created by @FabioBatSilva: Please Daniel Huss, A test case will be very useful.. |
Comment created by hussdl: failing test case |
Comment created by hussdl: Bug still present in release 2.3.3 Is someone taking care of this? It seems easy enough to fix. |
Comment created by @Ocramius: [~hussdl] you can open a PR with the attached patch + test. Can you do it or should I take care of it? |
Comment created by hussdl: Please take care of it for me this time. I'd rather avoid the git setup until I have a significant contribution to make. |
Comment created by @beberlei: This is invalid, you need to set the path expression differently, this is pseudo code:
|
Issue was closed with resolution "Invalid" |
Comment created by hussdl: Could this be bug in the DQL grammar, then? If I'm not mistaken, the resolution of an InExpression cannot involve an ArithmeticPrimary non-terminal. I find the correct solution as shown by Benjamin Eberlei to be quite astonishing, since a path expression like "alias.field" is not something I'd associate with the term "arithmetic". |
Jira issue originally created by user drak:
There is an undefined method called in Doctrine\ORM\Query\SqlWalker
Fatal error: Call to undefined method Doctrine\ORM\Query\AST\PathExpression::isSimpleArithmeticExpression() in vendor\doctrine\orm\lib\Doctrine\ORM\Query\SqlWalker.php on line 2091
The text was updated successfully, but these errors were encountered: