Merge pull request #7987 from beberlei/GH-7982-NoSqlExecutor

[GH-7982] no sql executor leads to parse error

Author: beberlei

lib/Doctrine/ORM/Query.php

@@ -133,7 +133,7 @@ final class Query extends AbstractQuery
      *
      * @var integer
      */
-    private $_state = self::STATE_CLEAN;
+    private $_state = self::STATE_DIRTY;
 
     /**
      * A snapshot of the parameter types the query was parsed with.

lib/Doctrine/ORM/Query/Parser.php

@@ -189,7 +189,7 @@ public function __construct(Query $query)
     {
         $this->query        = $query;
         $this->em           = $query->getEntityManager();
-        $this->lexer        = new Lexer($query->getDQL());
+        $this->lexer        = new Lexer((string) $query->getDQL());
         $this->parserResult = new ParserResult();
     }
 

tests/Doctrine/Tests/ORM/Query/LanguageRecognitionTest.php

@@ -134,6 +134,8 @@ public function invalidDQL()
             ['SELECT \'foo\' AS foo\\\\bar FROM Doctrine\Tests\Models\CMS\CmsUser u'],
             ['SELECT \'foo\' AS foo: FROM Doctrine\Tests\Models\CMS\CmsUser u'],
             ['SELECT \'foo\' AS foo:bar FROM Doctrine\Tests\Models\CMS\CmsUser u'],
+
+            ['0'],
         ];
     }
 

tests/Doctrine/Tests/ORM/Query/QueryTest.php

@@ -420,4 +420,13 @@ public function testValuesAreNotBeingResolvedForSpecifiedParameterTypes() : void
 
         self::assertEmpty($query->getResult());
     }
+
+    /** @group 7982 */
+    public function testNonExistentExecutor()
+    {
+        $this->expectException(QueryException::class);
+        $this->expectExceptionMessage('[Syntax Error] line 0, col -1: Error: Expected SELECT, UPDATE or DELETE, got end of string.');
+
+        $query = $this->_em->createQuery('0')->execute();
+    }
 }