Fix a Psalm complaint

Author: mpdude

lib/Doctrine/ORM/Query.php

@@ -149,8 +149,8 @@ class Query extends AbstractQuery
      */
     private $parserResult;
 
-    /** @var AbstractSqlExecutor */
-    private $sqlExecutor = null;
+    /** @var ?AbstractSqlExecutor */
+    private $sqlExecutor;
 
     /**
      * The first result to return (the "offset").

lib/Doctrine/ORM/Query/ParserResult.php

@@ -28,16 +28,16 @@ class ParserResult
     /**
      * The SQL executor used for executing the SQL.
      *
-     * @var AbstractSqlExecutor
+     * @var ?AbstractSqlExecutor
      */
     private $sqlExecutor;
 
     /**
      * The SQL executor used for executing the SQL.
      *
-     * @var SqlFinalizer
+     * @var ?SqlFinalizer
      */
-    private $sqlFinalizer = null;
+    private $sqlFinalizer;
 
     /**
      * The ResultSetMapping that describes how to map the SQL result set.
@@ -101,6 +101,10 @@ public function setSqlExecutor($executor)
      */
     public function getSqlExecutor()
     {
+        if ($this->sqlExecutor === null) {
+            throw new RuntimeException('This ParserResult was not created with an SqlExecutor');
+        }
+
         return $this->sqlExecutor;
     }
 

psalm-baseline.xml

@@ -1445,12 +1445,18 @@
     <InvalidArgument>
       <code>$sqlParams</code>
     </InvalidArgument>
+    <InvalidNullableReturnType>
+      <code>AbstractSqlExecutor</code>
+    </InvalidNullableReturnType>
     <LessSpecificReturnStatement>
-      <code><![CDATA[$this->parse()->getSqlExecutor()->getSqlStatements()]]></code>
+      <code><![CDATA[$this->getSqlExecutor()->getSqlStatements()]]></code>
     </LessSpecificReturnStatement>
     <MoreSpecificReturnType>
       <code><![CDATA[list<string>|string]]></code>
     </MoreSpecificReturnType>
+    <NullableReturnStatement>
+      <code><![CDATA[$this->sqlExecutor]]></code>
+    </NullableReturnStatement>
     <PossiblyNullArgument>
       <code><![CDATA[$this->getDQL()]]></code>
     </PossiblyNullArgument>
@@ -1866,6 +1872,15 @@
       <code><![CDATA[$this->_sqlStatements = &$this->sqlStatements]]></code>
     </UnsupportedPropertyReferenceUsage>
   </file>
+  <file src="lib/Doctrine/ORM/Query/Exec/FinalizedSelectExecutor.php">
+    <PossiblyInvalidArgument>
+      <code><![CDATA[$this->getSqlStatements()]]></code>
+    </PossiblyInvalidArgument>
+    <PropertyNotSetInConstructor>
+      <code>FinalizedSelectExecutor</code>
+      <code>FinalizedSelectExecutor</code>
+    </PropertyNotSetInConstructor>
+  </file>
   <file src="lib/Doctrine/ORM/Query/Exec/MultiTableDeleteExecutor.php">
     <InvalidReturnStatement>
       <code>$numDeleted</code>
@@ -2058,11 +2073,6 @@
       <code>$token === Lexer::T_IDENTIFIER</code>
     </RedundantConditionGivenDocblockType>
   </file>
-  <file src="lib/Doctrine/ORM/Query/ParserResult.php">
-    <PropertyNotSetInConstructor>
-      <code>$sqlExecutor</code>
-    </PropertyNotSetInConstructor>
-  </file>
   <file src="lib/Doctrine/ORM/Query/QueryExpressionVisitor.php">
     <InvalidReturnStatement>
       <code><![CDATA[new ArrayCollection($this->parameters)]]></code>