Use Connection::executeStatement() for "DELETE" statements

Author: phansys

CHANGELOG.md

@@ -20,6 +20,7 @@ a release.
 
 ## [Unreleased]
 ### Fixed
+- Tree: Check for affected rows at `ClosureTreeRepository::cleanUpClosure()` and `Closure::updateNode()`.
 - `Gedmo\Mapping\Driver\Xml::_loadMappingFile()` behavior in scenarios where `libxml_disable_entity_loader(true)` was previously
   called.
 - Loggable: Missing support for `versioned` fields at `attribute-override` in XML mapping.

src/Tree/Entity/Repository/ClosureTreeRepository.php

@@ -552,7 +552,7 @@ public function cleanUpClosure()
                 return $el['id'];
             }, $ids);
             $query = "DELETE FROM {$closureTableName} WHERE id IN (".implode(', ', $ids).')';
-            if (!$conn->executeQuery($query)) {
+            if (0 === $conn->executeStatement($query)) {
                 throw new \RuntimeException('Failed to remove incorrect closures');
             }
             $deletedClosuresCount += count($ids);

src/Tree/Strategy/ORM/Closure.php

@@ -439,7 +439,7 @@ public function updateNode(EntityManagerInterface $em, $node, $oldParent)
             if ([] !== $ids) {
                 // using subquery directly, sqlite acts unfriendly
                 $query = "DELETE FROM {$table} WHERE id IN (".implode(', ', $ids).')';
-                if (!$conn->executeQuery($query)) {
+                if (0 === $conn->executeStatement($query)) {
                     throw new RuntimeException('Failed to remove old closures');
                 }
             }