[EarlyReturn] Skip ChangeOrIfReturnToEarlyReturnRector on next node is null or return void (#796)

rules/EarlyReturn/Rector/If_/ChangeOrIfReturnToEarlyReturnRector.php

@@ -12,6 +12,7 @@ use PhpParser\Node\Stmt\If_;
 use PhpParser\Node\Stmt\Return_;
 use Rector\Core\NodeManipulator\IfManipulator;
 use Rector\Core\Rector\AbstractRector;
+use Rector\NodeTypeResolver\Node\AttributeKey;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 
@@ -89,6 +90,15 @@ CODE_SAMPLE
             return null;
         }
 
+        $nextNode = $node->getAttribute(AttributeKey::NEXT_NODE);
+        if ($nextNode === null) {
+            return null;
+        }
+
+        if ($nextNode instanceof Return_ && $nextNode->expr === null) {
+            return null;
+        }
+
         /** @var Return_ $return */
         $return = $node->stmts[0];
         $ifs = $this->createMultipleIfs($node->cond, $return, []);

tests/Issues/IssueEarlyReturnAndOrNarrow/Fixture/and_next_or.php.inc

@@ -31,12 +31,10 @@ class AndNextOrReturnVoid
      */
     public function run($a, $b, $c, $d)
     {
-        if ($a && $b) {
-            return null;
-        }
-        if ($c) {
+        if ($a && $b  || $c) {
             return null;
         }
+
         return;
     }
 }

tests/Issues/IssueReturnBeforeElseIf/Fixture/complex_if_cond_or_without_elseif.php.inc

@@ -28,10 +28,7 @@ class ComplexIfCondOrWithoutElseIf
 {
     public function run($a, $b, $c)
     {
-        if ($a && false === $b) {
-            return 'a';
-        }
-        if (! $c) {
+        if (($a && false === $b) || ! $c) {
             return 'a';
         }
         return 'b';