register rule

Author: TomasVotruba

rules-tests/DeadCode/Rector/Stmt/RemoveEmptyArrayConditionReturnRector/Fixture/allow_any_type.php.inc

@@ -0,0 +1,31 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\Stmt\RemoveEmptyArrayConditionReturnRector\Fixture;
+
+final class AllowAnyType
+{
+    public function run($result)
+    {
+        if ($result === []) {
+            return [];
+        }
+
+        return $result;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\Stmt\RemoveEmptyArrayConditionReturnRector\Fixture;
+
+final class AllowAnyType
+{
+    public function run($result)
+    {
+        return $result;
+    }
+}
+
+?>

rules/DeadCode/Rector/Stmt/RemoveEmptyArrayConditionReturnRector.php

@@ -9,6 +9,7 @@
 use PhpParser\Node\Expr\Array_;
 use PhpParser\Node\Expr\BinaryOp\Equal;
 use PhpParser\Node\Expr\BinaryOp\Identical;
+use PhpParser\Node\Stmt\If_;
 use PhpParser\Node\Stmt\Return_;
 use Rector\Contract\PhpParser\Node\StmtsAwareInterface;
 use Rector\Rector\AbstractRector;
@@ -74,7 +75,7 @@ public function refactor(Node $node): ?Node
         }
 
         foreach ($node->stmts as $key => $stmt) {
-            if (! $stmt instanceof Node\Stmt\If_) {
+            if (! $stmt instanceof If_) {
                 continue;
             }
 

src/Config/Level/DeadCodeLevel.php

@@ -53,6 +53,7 @@
 use Rector\DeadCode\Rector\PropertyProperty\RemoveNullPropertyInitializationRector;
 use Rector\DeadCode\Rector\Return_\RemoveDeadConditionAboveReturnRector;
 use Rector\DeadCode\Rector\StaticCall\RemoveParentCallWithoutParentRector;
+use Rector\DeadCode\Rector\Stmt\RemoveEmptyArrayConditionReturnRector;
 use Rector\DeadCode\Rector\Stmt\RemoveUnreachableStatementRector;
 use Rector\DeadCode\Rector\Switch_\RemoveDuplicatedCaseInSwitchRector;
 use Rector\DeadCode\Rector\Ternary\TernaryToBooleanOrFalseToBooleanAndRector;
@@ -124,6 +125,7 @@ final class DeadCodeLevel
         RemoveDeadCatchRector::class,
         RemoveDeadTryCatchRector::class,
         RemoveDeadIfForeachForRector::class,
+        RemoveEmptyArrayConditionReturnRector::class,
         RemoveDeadStmtRector::class,
         UnwrapFutureCompatibleIfPhpVersionRector::class,
         RemoveParentCallWithoutParentRector::class,

src/Testing/PHPUnit/AbstractLazyTestCase.php

@@ -70,6 +70,7 @@ private function includePreloadFilesAndScoperAutoload(): void
                 if (! class_exists(Version::class, true) || (int) Version::id() < 12) {
                     require_once __DIR__ . '/../../../preload.php';
                 }
+
                 // test case in rector split package
             } elseif (file_exists(__DIR__ . '/../../../../../../vendor')) {
                 require_once __DIR__ . '/../../../preload-split-package.php';