[CodeQuality] Fix identical boolcast

packages/CodeQuality/src/Rector/If_/SimplifyIfReturnBoolRector.php

@@ -7,6 +7,7 @@
 use Nette\Utils\Strings;
 use PhpParser\Node;
 use PhpParser\Node\Expr;
+use PhpParser\Node\Expr\BinaryOp;
 use PhpParser\Node\Expr\BinaryOp\Identical;
 use PhpParser\Node\Expr\BinaryOp\NotIdentical;
 use PhpParser\Node\Expr\BooleanNot;
@@ -189,11 +190,7 @@ private function boolCastOrNullCompareIfNeeded(Expr $expr): Expr
             }
         }
 
-        if ($expr instanceof BooleanNot) {
-            return $expr;
-        }
-
-        if ($this->isStaticType($expr, BooleanType::class)) {
+        if (! $this->isBoolCastNeeded($expr)) {
             return $expr;
         }
 
@@ -235,4 +232,17 @@ private function removeNullTypeFromUnionType(UnionType $unionType): Type
 
         return TypeFactoryStaticHelper::createUnionObjectType($unionedTypesWithoutNullType);
     }
+
+    private function isBoolCastNeeded(Expr $expr): bool
+    {
+        if ($expr instanceof BooleanNot) {
+            return false;
+        }
+
+        if ($this->isStaticType($expr, BooleanType::class)) {
+            return false;
+        }
+
+        return ! $expr instanceof BinaryOp;
+    }
 }

packages/CodeQuality/src/Rector/Include_/AbsolutizeRequireAndIncludePathRector.php

@@ -23,9 +23,11 @@ final class AbsolutizeRequireAndIncludePathRector extends AbstractRector
 {
     public function getDefinition(): RectorDefinition
     {
-        return new RectorDefinition('include/require to absolute path. This Rector might introduce backwards incompatible code, when the include/require beeing changed depends on the current working directory.', [
-            new CodeSample(
-                <<<'PHP'
+        return new RectorDefinition(
+            'include/require to absolute path. This Rector might introduce backwards incompatible code, when the include/require beeing changed depends on the current working directory.',
+            [
+                new CodeSample(
+                    <<<'PHP'
 class SomeClass
 {
     public function run()
@@ -37,7 +39,7 @@ public function run()
 }
 PHP
 ,
-                <<<'PHP'
+                    <<<'PHP'
 class SomeClass
 {
     public function run()
@@ -49,8 +51,9 @@ public function run()
 }
 PHP
 
-            ),
-        ]);
+                ),
+            ]
+        );
     }
 
     /**

packages/CodeQuality/tests/Rector/If_/SimplifyIfReturnBoolRector/Fixture/identical_non_cast.php.inc

@@ -0,0 +1,35 @@
+<?php
+
+namespace Rector\CodeQuality\Tests\Rector\If_\SimplifyIfReturnBoolRector\Fixture;
+
+use PhpParser\Node\Stmt\If_;
+
+class IdenticalNonCast
+{
+    public function run(If_ $if)
+    {
+        if ($if->else === null) {
+            return false;
+        }
+
+        return true;
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\CodeQuality\Tests\Rector\If_\SimplifyIfReturnBoolRector\Fixture;
+
+use PhpParser\Node\Stmt\If_;
+
+class IdenticalNonCast
+{
+    public function run(If_ $if)
+    {
+        return $if->else !== null;
+    }
+}
+
+?>

src/PhpParser/Node/Manipulator/IfManipulator.php

@@ -135,11 +135,7 @@ public function isEarlyElse(If_ $if): bool
             }
         }
 
-        if ($if->else === null) {
-            return false;
-        }
-
-        return true;
+        return $if->else !== null;
     }
 
     private function matchComparedAndReturnedNode(NotIdentical $notIdentical, Return_ $returnNode): ?Expr