[EarlyReturn] Decouple new rules (#4868)

This commit is contained in:
Tomas Votruba 2020-12-12 17:59:09 +01:00 committed by GitHub
parent f197e25793
commit eddb2b29c9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
63 changed files with 132 additions and 113 deletions

View File

@ -79,6 +79,7 @@
"Rector\\ConsoleDiffer\\": "packages/console-differ/src",
"Rector\\Core\\": "src",
"Rector\\DeadCode\\": "rules/dead-code/src",
"Rector\\EarlyReturn\\": "rules/early-return/src",
"Rector\\DeadDocBlock\\": "rules/dead-doc-block/src",
"Rector\\DoctrineAnnotationGenerated\\": "packages/doctrine-annotation-generated/src",
"Rector\\DoctrineCodeQuality\\": "rules/doctrine-code-quality/src",
@ -207,6 +208,7 @@
"Rector\\CodeQuality\\Tests\\": "rules/code-quality/tests",
"Rector\\CodingStyle\\Tests\\": "rules/coding-style/tests",
"Rector\\Core\\Tests\\": "tests",
"Rector\\EarlyReturn\\Tests\\": "rules/early-return/tests",
"Rector\\DeadCode\\Tests\\": "rules/dead-code/tests",
"Rector\\DoctrineCodeQuality\\Tests\\": "rules/doctrine-code-quality/tests",
"Rector\\DoctrineGedmoToKnplabs\\Tests\\": "rules/doctrine-gedmo-to-knplabs/tests",

View File

@ -2,10 +2,11 @@
declare(strict_types=1);
use Rector\SOLID\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector;
use Rector\SOLID\Rector\If_\ChangeAndIfToEarlyReturnRector;
use Rector\SOLID\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector;
use Rector\SOLID\Rector\If_\ChangeNestedIfsToEarlyReturnRector;
use Rector\EarlyReturn\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector;
use Rector\EarlyReturn\Rector\If_\ChangeAndIfToEarlyReturnRector;
use Rector\EarlyReturn\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector;
use Rector\EarlyReturn\Rector\If_\ChangeNestedIfsToEarlyReturnRector;
use Rector\EarlyReturn\Rector\If_\RemoveAlwaysElseRector;
use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
return static function (ContainerConfigurator $containerConfigurator): void {
@ -14,4 +15,5 @@ return static function (ContainerConfigurator $containerConfigurator): void {
$services->set(ChangeAndIfToEarlyReturnRector::class);
$services->set(ChangeIfElseValueAssignToEarlyReturnRector::class);
$services->set(ChangeNestedIfsToEarlyReturnRector::class);
$services->set(RemoveAlwaysElseRector::class);
};

View File

@ -7,7 +7,6 @@ use Rector\SOLID\Rector\Class_\ChangeReadOnlyVariableWithDefaultValueToConstantR
use Rector\SOLID\Rector\Class_\FinalizeClassesWithoutChildrenRector;
use Rector\SOLID\Rector\Class_\MakeUnusedClassesWithChildrenAbstractRector;
use Rector\SOLID\Rector\Class_\RepeatedLiteralToClassConstantRector;
use Rector\SOLID\Rector\If_\RemoveAlwaysElseRector;
use Rector\SOLID\Rector\Property\AddFalseDefaultToBoolPropertyRector;
use Rector\SOLID\Rector\Property\ChangeReadOnlyPropertyWithDefaultValueToConstantRector;
use Rector\SOLID\Rector\Variable\MoveVariableDeclarationNearReferenceRector;
@ -22,7 +21,6 @@ return static function (ContainerConfigurator $containerConfigurator): void {
$services->set(ChangeReadOnlyVariableWithDefaultValueToConstantRector::class);
$services->set(AddFalseDefaultToBoolPropertyRector::class);
$services->set(RepeatedLiteralToClassConstantRector::class);
$services->set(RemoveAlwaysElseRector::class);
$services->set(UseMessageVariableForSprintfInSymfonyStyleRector::class);
$services->set(MoveVariableDeclarationNearReferenceRector::class);
};

View File

@ -14096,7 +14096,7 @@ Add false default to bool properties, to prevent null compare errors
Changes if && to early return
- class: `Rector\SOLID\Rector\If_\ChangeAndIfToEarlyReturnRector`
- class: `Rector\EarlyReturn\Rector\If_\ChangeAndIfToEarlyReturnRector`
```diff
class SomeClass
@ -14125,7 +14125,7 @@ Changes if && to early return
Change if/else value to early return
- class: `Rector\SOLID\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector`
- class: `Rector\EarlyReturn\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector`
```diff
class SomeClass
@ -14151,7 +14151,7 @@ Change if/else value to early return
Change nested ifs to foreach with continue
- class: `Rector\SOLID\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector`
- class: `Rector\EarlyReturn\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector`
```diff
class SomeClass
@ -14184,7 +14184,7 @@ Change nested ifs to foreach with continue
Change nested ifs to early return
- class: `Rector\SOLID\Rector\If_\ChangeNestedIfsToEarlyReturnRector`
- class: `Rector\EarlyReturn\Rector\If_\ChangeNestedIfsToEarlyReturnRector`
```diff
class SomeClass
@ -14395,7 +14395,7 @@ return static function (ContainerConfigurator $containerConfigurator): void {
Split if statement, when if condition always break execution flow
- class: `Rector\SOLID\Rector\If_\RemoveAlwaysElseRector`
- class: `Rector\EarlyReturn\Rector\If_\RemoveAlwaysElseRector`
```diff
class SomeClass

View File

@ -0,0 +1,17 @@
<?php
declare(strict_types=1);
use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
return static function (ContainerConfigurator $containerConfigurator): void {
$services = $containerConfigurator->services();
$services->defaults()
->public()
->autowire()
->autoconfigure();
$services->load('Rector\EarlyReturn\\', __DIR__ . '/../src')
->exclude([__DIR__ . '/../src/Rector']);
};

View File

@ -2,7 +2,7 @@
declare(strict_types=1);
namespace Rector\SOLID\NodeTransformer;
namespace Rector\EarlyReturn\NodeTransformer;
use PhpParser\Node\Expr;
use PhpParser\Node\Expr\BinaryOp;

View File

@ -2,7 +2,7 @@
declare(strict_types=1);
namespace Rector\SOLID\Rector\Foreach_;
namespace Rector\EarlyReturn\Rector\Foreach_;
use PhpParser\Node;
use PhpParser\Node\Expr;
@ -16,13 +16,13 @@ use PhpParser\Node\Stmt\Foreach_;
use PhpParser\Node\Stmt\If_;
use Rector\Core\PhpParser\Node\Manipulator\IfManipulator;
use Rector\Core\Rector\AbstractRector;
use Rector\EarlyReturn\NodeTransformer\ConditionInverter;
use Rector\NodeTypeResolver\Node\AttributeKey;
use Rector\SOLID\NodeTransformer\ConditionInverter;
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
* @see \Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\ChangeNestedForeachIfsToEarlyContinueRectorTest
* @see \Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\ChangeNestedForeachIfsToEarlyContinueRectorTest
*/
final class ChangeNestedForeachIfsToEarlyContinueRector extends AbstractRector
{

View File

@ -2,7 +2,7 @@
declare(strict_types=1);
namespace Rector\SOLID\Rector\If_;
namespace Rector\EarlyReturn\Rector\If_;
use PhpParser\Node;
use PhpParser\Node\Expr;
@ -20,13 +20,13 @@ use PhpParser\Node\Stmt\While_;
use Rector\Core\PhpParser\Node\Manipulator\IfManipulator;
use Rector\Core\PhpParser\Node\Manipulator\StmtsManipulator;
use Rector\Core\Rector\AbstractRector;
use Rector\EarlyReturn\NodeTransformer\ConditionInverter;
use Rector\NodeTypeResolver\Node\AttributeKey;
use Rector\SOLID\NodeTransformer\ConditionInverter;
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
* @see \Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\ChangeAndIfToEarlyReturnRectorTest
* @see \Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\ChangeAndIfToEarlyReturnRectorTest
*/
final class ChangeAndIfToEarlyReturnRector extends AbstractRector
{

View File

@ -2,7 +2,7 @@
declare(strict_types=1);
namespace Rector\SOLID\Rector\If_;
namespace Rector\EarlyReturn\Rector\If_;
use PhpParser\Node;
use PhpParser\Node\Expr\Assign;
@ -18,7 +18,7 @@ use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
* @see https://engineering.helpscout.com/reducing-complexity-with-guard-clauses-in-php-and-javascript-74600fd865c7
*
* @see \Rector\SOLID\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\ChangeIfElseValueAssignToEarlyReturnRectorTest
* @see \Rector\EarlyReturn\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\ChangeIfElseValueAssignToEarlyReturnRectorTest
*/
final class ChangeIfElseValueAssignToEarlyReturnRector extends AbstractRector
{

View File

@ -2,7 +2,7 @@
declare(strict_types=1);
namespace Rector\SOLID\Rector\If_;
namespace Rector\EarlyReturn\Rector\If_;
use PhpParser\Node;
use PhpParser\Node\Expr\BinaryOp\BooleanAnd;
@ -11,13 +11,13 @@ use PhpParser\Node\Stmt\If_;
use PhpParser\Node\Stmt\Return_;
use Rector\Core\PhpParser\Node\Manipulator\IfManipulator;
use Rector\Core\Rector\AbstractRector;
use Rector\EarlyReturn\NodeTransformer\ConditionInverter;
use Rector\NodeTypeResolver\Node\AttributeKey;
use Rector\SOLID\NodeTransformer\ConditionInverter;
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
* @see \Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\ChangeNestedIfsToEarlyReturnRectorTest
* @see \Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\ChangeNestedIfsToEarlyReturnRectorTest
*/
final class ChangeNestedIfsToEarlyReturnRector extends AbstractRector
{

View File

@ -2,7 +2,7 @@
declare(strict_types=1);
namespace Rector\SOLID\Rector\If_;
namespace Rector\EarlyReturn\Rector\If_;
use PhpParser\Node;
use PhpParser\Node\Expr\Exit_;
@ -18,7 +18,7 @@ use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
* @see \Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElseRector\RemoveAlwaysElseRectorTest
* @see \Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElseRector\RemoveAlwaysElseRectorTest
*/
final class RemoveAlwaysElseRector extends AbstractRector
{

View File

@ -2,10 +2,10 @@
declare(strict_types=1);
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector;
use Iterator;
use Rector\SOLID\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector;
use Rector\EarlyReturn\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector;
use Rector\Testing\PHPUnit\AbstractRectorTestCase;
use Symplify\SmartFileSystem\SmartFileInfo;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
class CommentInsideIfStatement
{
@ -23,7 +23,7 @@ class CommentInsideIfStatement
-----
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
class CommentInsideIfStatement
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
class SomeClass
{
@ -22,7 +22,7 @@ class SomeClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
class MultiExprs
{
@ -24,7 +24,7 @@ class MultiExprs
-----
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
class MultiExprs
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
use Rector\Testing\Contract\RunnableInterface;
@ -30,7 +30,7 @@ class MultiExprsWithOrBothTruthy implements RunnableInterface
-----
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
use Rector\Testing\Contract\RunnableInterface;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
use Rector\Testing\Contract\RunnableInterface;
@ -30,7 +30,7 @@ class MultiExprsWithBoolOprnds implements RunnableInterface
-----
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
use Rector\Testing\Contract\RunnableInterface;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
class SkipSingleLine
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\Foreach_\ChangeNestedForeachIfsToEarlyContinueRector\Fixture;
class SkipText
{

View File

@ -2,10 +2,10 @@
declare(strict_types=1);
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector;
use Iterator;
use Rector\SOLID\Rector\If_\ChangeAndIfToEarlyReturnRector;
use Rector\EarlyReturn\Rector\If_\ChangeAndIfToEarlyReturnRector;
use Rector\Testing\PHPUnit\AbstractRectorTestCase;
use Symplify\SmartFileSystem\SmartFileInfo;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class BooleanNotClass
{
@ -18,7 +18,7 @@ class BooleanNotClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class BooleanNotClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
use PhpParser\Node;
use PhpParser\Node\Identifier;
@ -26,7 +26,7 @@ class ClosureClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
use PhpParser\Node;
use PhpParser\Node\Identifier;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
use PhpParser\Node\Identifier;
use PhpParser\Node\Name;
@ -21,7 +21,7 @@ class ContinueInForeachClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
use PhpParser\Node\Identifier;
use PhpParser\Node\Name;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class EqualsClass
{
@ -18,7 +18,7 @@ class EqualsClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class EqualsClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class SomeClass
{
@ -18,7 +18,7 @@ class SomeClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
use PhpParser\Node\Identifier;
use PhpParser\Node\Name;
@ -25,7 +25,7 @@ class IfInForeachClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
use PhpParser\Node\Identifier;
use PhpParser\Node\Name;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class KeepCommentOfRemovedNodeClass
{
@ -19,7 +19,7 @@ class KeepCommentOfRemovedNodeClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class KeepCommentOfRemovedNodeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class MoreThanTwoConditionsClass
{
@ -16,7 +16,7 @@ class MoreThanTwoConditionsClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class MoreThanTwoConditionsClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class NonFirstLevelIfClass
{
@ -18,7 +18,7 @@ class NonFirstLevelIfClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class NonFirstLevelIfClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class OnlyOneIfBeforeClassMethodReturnClass
{
@ -22,7 +22,7 @@ class OnlyOneIfBeforeClassMethodReturnClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class OnlyOneIfBeforeClassMethodReturnClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class SkipConditionWithOr
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
use PhpParser\Node\Identifier;
use PhpParser\Node\Name;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class SkipNonVoidReturnClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class SkipNotBeforeFirstLevelReturnClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class SkipParentIfWithVoidReturn
{

View File

@ -1,7 +1,7 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class SkipElseElseIfClass

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeAndIfToEarlyReturnRector\Fixture;
class SkipWithVoidReturn
{

View File

@ -2,10 +2,10 @@
declare(strict_types=1);
namespace Rector\SOLID\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector;
use Iterator;
use Rector\SOLID\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector;
use Rector\EarlyReturn\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector;
use Rector\Testing\PHPUnit\AbstractRectorTestCase;
use Symplify\SmartFileSystem\SmartFileInfo;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\Fixture;
class SomeClass
{
@ -20,7 +20,7 @@ class SomeClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\Fixture;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\Fixture;
class LostComment
{
@ -22,7 +22,7 @@ class LostComment
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeIfElseValueAssignToEarlyReturnRector\Fixture;
class LostComment
{

View File

@ -2,10 +2,10 @@
declare(strict_types=1);
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector;
use Iterator;
use Rector\SOLID\Rector\If_\ChangeNestedIfsToEarlyReturnRector;
use Rector\EarlyReturn\Rector\If_\ChangeNestedIfsToEarlyReturnRector;
use Rector\Testing\PHPUnit\AbstractRectorTestCase;
use Symplify\SmartFileSystem\SmartFileInfo;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
use Rector\NodeTypeResolver\Node\AttributeKey;
@ -23,7 +23,7 @@ class BooleanAnd
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
use Rector\NodeTypeResolver\Node\AttributeKey;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
class SomeClass
{
@ -20,7 +20,7 @@ class SomeClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
class HavingElseIfs
{
@ -22,7 +22,7 @@ class HavingElseIfs
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
class HavingElseIfs
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
use PhpParser\Node\Stmt\If_;
@ -40,7 +40,7 @@ class RectorSample
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
use PhpParser\Node\Stmt\If_;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
class SkipSingleIf
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
class ThreeNesting
{
@ -22,7 +22,7 @@ class ThreeNesting
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\ChangeNestedIfsToEarlyReturnRector\Fixture;
class ThreeNesting
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class EarlyElse
{
@ -18,7 +18,7 @@ class EarlyElse
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class EarlyElse
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\ElseIf_;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\ElseIf_;
class SomeClass
{
@ -22,7 +22,7 @@ class SomeClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\ElseIf_;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\ElseIf_;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\EmptyIf;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\EmptyIf;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Exit_;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Exit_;
class SomeClass
{
@ -19,7 +19,7 @@ class SomeClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Exit_;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Exit_;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class SomeClass
{
@ -18,7 +18,7 @@ class SomeClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class ForContinue
{
@ -33,7 +33,7 @@ class ForContinue
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class ForContinue
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class FunctionForeachContinue
{
@ -22,7 +22,7 @@ class FunctionForeachContinue
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class FunctionForeachContinue
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class LostCommentBeforeElseIf
{
@ -27,7 +27,7 @@ class LostCommentBeforeElseIf
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class LostCommentBeforeElseIf
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\NoBreak;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\NoBreak;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class ProcessEmptyReturnLast
{
@ -34,7 +34,7 @@ class ProcessEmptyReturnLast
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class ProcessEmptyReturnLast
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\SimpleIf;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\SimpleIf;
class SomeClass
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
use Throwable;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
use PhpParser\Node\Expr\BinaryOp\Equal;

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Fixture;
class SkipNotOnlyElse
{

View File

@ -1,6 +1,6 @@
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Throw_;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Throw_;
class SomeClass
{
@ -19,7 +19,7 @@ class SomeClass
-----
<?php
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElse\Throw_;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElse\Throw_;
class SomeClass
{

View File

@ -2,10 +2,10 @@
declare(strict_types=1);
namespace Rector\SOLID\Tests\Rector\If_\RemoveAlwaysElseRector;
namespace Rector\EarlyReturn\Tests\Rector\If_\RemoveAlwaysElseRector;
use Iterator;
use Rector\SOLID\Rector\If_\RemoveAlwaysElseRector;
use Rector\EarlyReturn\Rector\If_\RemoveAlwaysElseRector;
use Rector\Testing\PHPUnit\AbstractRectorTestCase;
use Symplify\SmartFileSystem\SmartFileInfo;