[EarlyReturn] Remove ReturnBinaryAndToEarlyReturnRector as creates co…

…de hard to read and flips condition (#4478)

build/target-repository/docs/rector_rules_overview.md

@@ -1,4 +1,4 @@
-# 362 Rules Overview
+# 361 Rules Overview
 
 <br>
 
@@ -12,7 +12,7 @@
 
 - [DeadCode](#deadcode) (42)
 
-- [EarlyReturn](#earlyreturn) (10)
+- [EarlyReturn](#earlyreturn) (9)
 
 - [Naming](#naming) (6)
 
@@ -3377,29 +3377,6 @@ Split if statement, when if condition always break execution flow
 
 <br>
 
-### ReturnBinaryAndToEarlyReturnRector
-
-Changes Single return of && to early returns
-
-- class: [`Rector\EarlyReturn\Rector\Return_\ReturnBinaryAndToEarlyReturnRector`](../rules/EarlyReturn/Rector/Return_/ReturnBinaryAndToEarlyReturnRector.php)
-
-```diff
- class SomeClass
- {
-     public function accept()
-     {
--        return $this->something() && $this->somethingelse();
-+        if (! $this->something()) {
-+            return false;
-+        }
-+
-+        return (bool) $this->somethingelse();
-     }
- }
-```
-
-<br>
-
 ### ReturnBinaryOrToEarlyReturnRector
 
 Changes Single return of || to early returns

config/set/early-return.php

@@ -10,7 +10,6 @@
 use Rector\EarlyReturn\Rector\If_\ChangeOrIfContinueToMultiContinueRector;
 use Rector\EarlyReturn\Rector\If_\RemoveAlwaysElseRector;
 use Rector\EarlyReturn\Rector\Return_\PreparedValueToEarlyReturnRector;
-use Rector\EarlyReturn\Rector\Return_\ReturnBinaryAndToEarlyReturnRector;
 use Rector\EarlyReturn\Rector\Return_\ReturnBinaryOrToEarlyReturnRector;
 use Rector\EarlyReturn\Rector\StmtsAwareInterface\ReturnEarlyIfVariableRector;
 
@@ -21,7 +20,6 @@
         ChangeIfElseValueAssignToEarlyReturnRector::class,
         ChangeNestedIfsToEarlyReturnRector::class,
         RemoveAlwaysElseRector::class,
-        ReturnBinaryAndToEarlyReturnRector::class,
         ChangeOrIfContinueToMultiContinueRector::class,
         PreparedValueToEarlyReturnRector::class,
         ReturnBinaryOrToEarlyReturnRector::class,