Fix Psalm complaints after merge-up

mpdude · Feb 1, 2024 · c7a91a4 · c7a91a4
1 parent 1df03f2
commit c7a91a4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/psalm-baseline.xml b/psalm-baseline.xml
@@ -890,9 +890,6 @@
     <PossiblyInvalidIterator>
       <code><![CDATA[$this->sqlStatements]]></code>
     </PossiblyInvalidIterator>
-    <PropertyNotSetInConstructor>
-      <code>MultiTableUpdateExecutor</code>
-    </PropertyNotSetInConstructor>
   </file>
   <file src="src/Query/Exec/SingleSelectExecutor.php">
     <PossiblyInvalidArgument>

diff --git a/psalm.xml b/psalm.xml
@@ -107,7 +107,6 @@
             <errorLevel type="suppress">
                 <!-- https://github.com/vimeo/psalm/issues/8819 -->
                 <file name="src/Internal/Hydration/AbstractHydrator.php"/>
-                <referencedClass name="Doctrine\DBAL\Platforms\MySQLPlatform" />
             </errorLevel>
         </InvalidReturnType>
         <InvalidParamDefault>

diff --git a/src/Query/Exec/MultiTableUpdateExecutor.php b/src/Query/Exec/MultiTableUpdateExecutor.php
@@ -42,10 +42,11 @@ class MultiTableUpdateExecutor extends AbstractSqlExecutor
      */
     public function __construct(AST\Node $AST, SqlWalker $sqlWalker)
     {
-        $em            = $sqlWalker->getEntityManager();
-        $conn          = $em->getConnection();
-        $platform      = $conn->getDatabasePlatform();
-        $quoteStrategy = $em->getConfiguration()->getQuoteStrategy();
+        $em                  = $sqlWalker->getEntityManager();
+        $conn                = $em->getConnection();
+        $platform            = $conn->getDatabasePlatform();
+        $quoteStrategy       = $em->getConfiguration()->getQuoteStrategy();
+        $this->sqlStatements = [];
 
         if ($conn instanceof PrimaryReadReplicaConnection) {
             $conn->ensureConnectedToPrimary();