[Dbal40] Skip different object on ExecuteQueryParamsToBindValueRector

rules-tests/Dbal40/Rector/StmtsAwareInterface/ExecuteQueryParamsToBindValueRector/Fixture/some_different_object_with_execute.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Doctrine\Tests\Dbal40\Rector\StmtsAwareInterface\ExecuteQueryParamsToBindValueRector\Fixture;
+
+use Rector\Doctrine\Tests\Dbal40\Rector\StmtsAwareInterface\ExecuteQueryParamsToBindValueRector\Source\Statement;
+
+final class SkipDifferentObjectWithExecute
+{
+    public function run(Statement $statement, array $params): void
+    {
+        $result = $statement->executeQuery($params);
+    }
+}

rules-tests/Dbal40/Rector/StmtsAwareInterface/ExecuteQueryParamsToBindValueRector/Source/Statement.php

@@ -0,0 +1,9 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Doctrine\Tests\Dbal40\Rector\StmtsAwareInterface\ExecuteQueryParamsToBindValueRector\Source;
+
+class Statement
+{
+}

rules/Dbal40/Rector/StmtsAwareInterface/ExecuteQueryParamsToBindValueRector.php

@@ -4,16 +4,17 @@
 
 namespace Rector\Doctrine\Dbal40\Rector\StmtsAwareInterface;
 
-use PhpParser\Node\Expr\Variable;
-use PhpParser\Node\Arg;
-use PhpParser\Node\Expr\BinaryOp\Plus;
-use PhpParser\Node\Scalar\LNumber;
 use PhpParser\Node;
+use PhpParser\Node\Arg;
 use PhpParser\Node\Expr;
+use PhpParser\Node\Expr\BinaryOp\Plus;
 use PhpParser\Node\Expr\MethodCall;
+use PhpParser\Node\Expr\Variable;
+use PhpParser\Node\Scalar\LNumber;
 use PhpParser\Node\Stmt\Expression;
 use PhpParser\Node\Stmt\Foreach_;
 use PhpParser\NodeFinder;
+use PHPStan\Type\ObjectType;
 use Rector\Contract\PhpParser\Node\StmtsAwareInterface;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
@@ -72,17 +73,21 @@ public function refactor(Node $node): ?StmtsAwareInterface
         $nodeFinder = new NodeFinder();
 
         $hasChanged = false;
-
+        $objectType = new ObjectType('Doctrine\DBAL\Statement');
         foreach ((array) $node->stmts as $key => $stmt) {
             if (! $stmt instanceof Expression) {
                 continue;
             }
 
-            $executeQueryMethodCall = $nodeFinder->findFirst($stmt, function (Node $node): bool {
+            $executeQueryMethodCall = $nodeFinder->findFirst($stmt, function (Node $node) use ($objectType): bool {
                 if (! $node instanceof MethodCall) {
                     return false;
                 }
 
+                if (! $this->isObjectType($node->var, $objectType)) {
+                    return false;
+                }
+
                 if (! $this->isName($node->name, 'executeQuery')) {
                     return false;
                 }
@@ -95,7 +100,8 @@ public function refactor(Node $node): ?StmtsAwareInterface
             }
 
             // remove args
-            $stmtsExpr = $executeQueryMethodCall->getArgs()[0]->value;
+            $stmtsExpr = $executeQueryMethodCall->getArgs()[0]
+                ->value;
             $executeQueryMethodCall->args = [];
 
             $hasChanged = true;