[CodeQuality][EarlyReturn] Handle ChangeOrIfReturnToEarlyReturnRector + ChangeAndIfToEarlyReturnRector + DateTimeToDateTimeInterfaceRector

[samsonasik]

Given the following code:

class AndNextOr
{
    public function run($a, $b, $c, $d)
    {
        if ($a && $b || $c) {
            return null;
        }

        $a = 1;
        $b = 2;

        if ($b > $a) {
            return null;
        }

        $d = 1;

        return;
    }
}

Currently produce repetitive ifs and and returns:

+        if (!$a) {
+            return;
+        }
+        if (!$b) {
+            return;
+        }
+        if (!$a) {
+            return;
+        }
+        if (!$b) {
+            return;
+        }
+        if (!$a) {
+            return;
+        }
+        if (!$b) {
+            return;
+        }
+        if (!$a) {
+            return;
+        }
+        if (!$b) {
+            return;
+        }
+        if (!$a) {
+            return;
+        }
+        if (!$b) {
+            return;
+        }
+        if (!$a) {
+            return;
+        }
+        if (!$b) {
+            return;
+        }
+        if (!$a) {
+            return;
+        }
+        if (!$b) {
+            return;
+        }
+        if (!$a) {
+            return;
+        }
+        if (!$b) {
+            return;
+        }
         if ($a && $b) {
             return null;
         }
+        return null;
+        return null;
+        return null;
+        return null;
+        return null;
+        return null;
+        return null;
+        return null;
         if ($c) {
             return null;
         }

Used rules:

Rector\CodeQuality\Rector\ClassMethod\DateTimeToDateTimeInterfaceRector;
Rector\EarlyReturn\Rector\If_\ChangeAndIfToEarlyReturnRector;
Rector\EarlyReturn\Rector\If_\ChangeOrIfReturnToEarlyReturnRector;

[samsonasik]

Fixed 🎉

[samsonasik]

All checks have passed 🎉 @TomasVotruba it is ready for review.