Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Invalid
    • Affects Version/s: 2.3
    • Fix Version/s: None
    • Component/s: ORM
    • Labels:
      None

      Description

      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

      1. SqlWalkerTest.php
        1 kB
        Daniel Huss
      2. SqlWalker.patch
        0.7 kB
        Daniel Huss

        Activity

        Hide
        Daniel Huss added a comment -

        Bug still present in release 2.3.3

        Is someone taking care of this? It seems easy enough to fix.

        Show
        Daniel Huss added a comment - Bug still present in release 2.3.3 Is someone taking care of this? It seems easy enough to fix.
        Hide
        Marco Pivetta added a comment -

        Daniel Huss you can open a PR with the attached patch + test. Can you do it or should I take care of it?

        Show
        Marco Pivetta added a comment - Daniel Huss you can open a PR with the attached patch + test. Can you do it or should I take care of it?
        Hide
        Daniel Huss added a comment -

        Please take care of it for me this time. I'd rather avoid the git setup until I have a significant contribution to make.

        Show
        Daniel Huss added a comment - Please take care of it for me this time. I'd rather avoid the git setup until I have a significant contribution to make.
        Hide
        Benjamin Eberlei added a comment -

        This is invalid, you need to set the path expression differently, this is pseudo code:

        new ArtihmeticExpression(new SimpleArithmeticExpression(arithmeticsTerms => new PathExpression)));
        
        Show
        Benjamin Eberlei added a comment - This is invalid, you need to set the path expression differently, this is pseudo code: new ArtihmeticExpression( new SimpleArithmeticExpression(arithmeticsTerms => new PathExpression)));
        Hide
        Daniel Huss added a comment -

        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".

        Show
        Daniel Huss added a comment - 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".

          People

          • Assignee:
            Benjamin Eberlei
            Reporter:
            Karma Dordrak (Drak)
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: