diff --git a/rector.php b/rector.php index b24f7fe82f0..0d81cf84a09 100644 --- a/rector.php +++ b/rector.php @@ -18,7 +18,6 @@ naming: true, instanceOf: true, earlyReturn: true, - strictBooleans: true, rectorPreset: true, phpunitCodeQuality: true ) diff --git a/rules-tests/Php80/Rector/Class_/ClassPropertyAssignToConstructorPromotionRector/Fixture/do_not_remove_parameter_attribute.php.inc b/rules-tests/Php80/Rector/Class_/ClassPropertyAssignToConstructorPromotionRector/Fixture/do_not_remove_parameter_attribute.php.inc index dfb36b5303a..ddf318dcaaf 100644 --- a/rules-tests/Php80/Rector/Class_/ClassPropertyAssignToConstructorPromotionRector/Fixture/do_not_remove_parameter_attribute.php.inc +++ b/rules-tests/Php80/Rector/Class_/ClassPropertyAssignToConstructorPromotionRector/Fixture/do_not_remove_parameter_attribute.php.inc @@ -25,7 +25,7 @@ namespace Rector\Tests\Php80\Rector\Class_\ClassPropertyAssignToConstructorPromo final class DoNotRemoveParameterAttribute { public function __construct( - #[\SensitiveParameter]private string $password + #[\SensitiveParameter] private string $password ) { } diff --git a/rules/Php82/NodeManipulator/ReadonlyClassManipulator.php b/rules/Php82/NodeManipulator/ReadonlyClassManipulator.php index b28f3b74a2b..2a56ca4aacb 100644 --- a/rules/Php82/NodeManipulator/ReadonlyClassManipulator.php +++ b/rules/Php82/NodeManipulator/ReadonlyClassManipulator.php @@ -225,8 +225,8 @@ private function shouldSkipClass(Class_ $class): bool } return $class->extends instanceof FullyQualified && ! $this->reflectionProvider->hasClass( - $class->extends->toString() - ); + $class->extends->toString() + ); } /** diff --git a/rules/Php83/Rector/Class_/ReadOnlyAnonymousClassRector.php b/rules/Php83/Rector/Class_/ReadOnlyAnonymousClassRector.php index a64bff2986e..f3ac0a0cdd9 100644 --- a/rules/Php83/Rector/Class_/ReadOnlyAnonymousClassRector.php +++ b/rules/Php83/Rector/Class_/ReadOnlyAnonymousClassRector.php @@ -68,7 +68,7 @@ public function refactor(Node $node): ?Node return null; } - return $this->readonlyClassManipulator->process($node, $this->file); + return $this->readonlyClassManipulator->process($node); } public function provideMinPhpVersion(): int diff --git a/rules/Privatization/Rector/Class_/FinalizeTestCaseClassRector.php b/rules/Privatization/Rector/Class_/FinalizeTestCaseClassRector.php index ab709b7feb1..0a8db64329d 100644 --- a/rules/Privatization/Rector/Class_/FinalizeTestCaseClassRector.php +++ b/rules/Privatization/Rector/Class_/FinalizeTestCaseClassRector.php @@ -7,7 +7,6 @@ use PhpParser\Node; use PhpParser\Node\Stmt\Class_; use PHPStan\Reflection\ReflectionProvider; -use Rector\Php81\NodeManipulator\AttributeGroupNewLiner; use Rector\Privatization\NodeManipulator\VisibilityManipulator; use Rector\Rector\AbstractRector; use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample; @@ -20,8 +19,7 @@ final class FinalizeTestCaseClassRector extends AbstractRector { public function __construct( private readonly ReflectionProvider $reflectionProvider, - private readonly VisibilityManipulator $visibilityManipulator, - private readonly AttributeGroupNewLiner $attributeGroupNewLiner + private readonly VisibilityManipulator $visibilityManipulator ) { } @@ -85,10 +83,6 @@ public function refactor(Node $node): ?Node return null; } - if ($node->attrGroups !== []) { - $this->attributeGroupNewLiner->newLine($this->file, $node); - } - $this->visibilityManipulator->makeFinal($node); return $node;