[DeadCode] Skip on object with __destruct method on RemoveUnusedVariableAssignRector (#7711)

* [DeadCode] Skip on object with __destruct method on RemoveUnusedVariableAssignRector

* [ci-review] Rector Rectify

---------

Co-authored-by: GitHub Action <[email protected]>

Author: samsonasik

rules-tests/DeadCode/Rector/Assign/RemoveUnusedVariableAssignRector/Fixture/skip_object_with_destruct_method.php.inc

@@ -0,0 +1,20 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\Assign\RemoveUnusedVariableAssignRector\Fixture;
+
+use Rector\Tests\DeadCode\Rector\Assign\RemoveUnusedVariableAssignRector\Source\SomeLock;
+
+final class SkipObjectWithDestructMethod
+{
+    public function run()
+    {
+        $lock = $this->createLock();
+
+        echo 'foobar';
+    }
+
+    private function createLock(): SomeLock
+    {
+        return new SomeLock();
+    }
+}

rules-tests/DeadCode/Rector/Assign/RemoveUnusedVariableAssignRector/Source/SomeLock.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\DeadCode\Rector\Assign\RemoveUnusedVariableAssignRector\Source;
+
+class SomeLock
+{
+    public function __destruct()
+    {
+        // Do important things
+    }
+}

rules/DeadCode/Rector/Assign/RemoveUnusedVariableAssignRector.php

@@ -19,13 +19,16 @@
 use PhpParser\Node\Stmt\Expression;
 use PhpParser\Node\Stmt\Function_;
 use PhpParser\NodeVisitor;
+use PHPStan\Reflection\ClassReflection;
+use PHPStan\Type\ObjectType;
 use Rector\DeadCode\SideEffect\SideEffectNodeDetector;
 use Rector\NodeAnalyzer\VariableAnalyzer;
 use Rector\NodeManipulator\StmtsManipulator;
 use Rector\Php\ReservedKeywordAnalyzer;
 use Rector\PhpParser\Enum\NodeGroup;
 use Rector\PhpParser\Node\BetterNodeFinder;
 use Rector\Rector\AbstractRector;
+use Rector\ValueObject\MethodName;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 
@@ -107,6 +110,10 @@ public function refactor(Node $node): null|ClassMethod|Function_
             /** @var Assign $assign */
             $assign = $currentStmt->expr;
 
+            if ($this->isObjectWithDestructMethod($assign->expr)) {
+                continue;
+            }
+
             if ($this->hasCallLikeInAssignExpr($assign)) {
                 // clean safely
                 $cleanAssignedExpr = $this->cleanCastedExpr($assign->expr);
@@ -127,6 +134,21 @@ public function refactor(Node $node): null|ClassMethod|Function_
         return null;
     }
 
+    private function isObjectWithDestructMethod(Expr $expr): bool
+    {
+        $exprType = $this->getType($expr);
+        if (! $exprType instanceof ObjectType) {
+            return false;
+        }
+
+        $classReflection = $exprType->getClassReflection();
+        if (! $classReflection instanceof ClassReflection) {
+            return false;
+        }
+
+        return $classReflection->hasNativeMethod(MethodName::DESTRUCT);
+    }
+
     private function cleanCastedExpr(Expr $expr): Expr
     {
         if (! $expr instanceof Cast) {