[automated] Apply Coding Standard (#4621)

Co-authored-by: TomasVotruba <TomasVotruba@users.noreply.github.com>
rectorphp · Jul 30, 2023 · 8c62386 · 8c62386
1 parent 49f2473
commit 8c62386
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 13 deletions.
diff --git a/rules/TypeDeclaration/Rector/ClassMethod/AddMethodCallBasedStrictParamTypeRector.php b/rules/TypeDeclaration/Rector/ClassMethod/AddMethodCallBasedStrictParamTypeRector.php
@@ -4,8 +4,8 @@
 
 namespace Rector\TypeDeclaration\Rector\ClassMethod;
 
-use PhpParser\Node\Name;
 use PhpParser\Node;
+use PhpParser\Node\Name;
 use PhpParser\Node\Stmt\Class_;
 use PhpParser\Node\Stmt\ClassMethod;
 use Rector\Core\PhpParser\NodeFinder\LocalMethodCallFinder;
@@ -91,16 +91,17 @@ public function refactor(Node $node): ?Node
 
             $isPrivate =
                 ($node->isFinal()
-                && !$node->extends instanceof Name
+                && ! $node->extends instanceof Name
                 && $node->implements === []
                 && $method->isProtected())
-                || ($method->isFinal()
-                    && !$node->extends instanceof Name
+                || (
+                    $method->isFinal()
+                    && ! $node->extends instanceof Name
                     && $node->implements === []
                 )
                 || $method->isPrivate();
 
-            if (!$isPrivate) {
+            if (! $isPrivate) {
                 continue;
             }
 

diff --git a/src/NodeManipulator/PropertyManipulator.php b/src/NodeManipulator/PropertyManipulator.php
@@ -4,7 +4,6 @@
 
 namespace Rector\Core\NodeManipulator;
 
-use PHPStan\Analyser\Scope;
 use Doctrine\ORM\Mapping\Table;
 use PhpParser\Node;
 use PhpParser\Node\Expr\PropertyFetch;
@@ -14,6 +13,7 @@
 use PhpParser\Node\Stmt\ClassMethod;
 use PhpParser\Node\Stmt\Property;
 use PhpParser\Node\Stmt\Trait_;
+use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\ClassReflection;
 use PHPStan\Type\ObjectType;
 use Rector\BetterPhpDocParser\PhpDocInfo\PhpDocInfo;
@@ -177,7 +177,6 @@ private function isChangeableContext(PropertyFetch | StaticPropertyFetch $proper
         return $propertyFetch->getAttribute(AttributeKey::IS_USED_AS_ARG_BY_REF_VALUE, false) === true;
     }
 
-
     private function hasAllowedNotReadonlyAnnotationOrAttribute(PhpDocInfo $phpDocInfo, Class_ $class): bool
     {
         if ($phpDocInfo->hasByAnnotationClasses(self::DOCTRINE_PROPERTY_ANNOTATIONS)) {

diff --git a/src/PhpParser/NodeFinder/PropertyFetchFinder.php b/src/PhpParser/NodeFinder/PropertyFetchFinder.php
@@ -4,7 +4,6 @@
 
 namespace Rector\Core\PhpParser\NodeFinder;
 
-use PHPStan\Type\StaticType;
 use PhpParser\Node;
 use PhpParser\Node\Expr;
 use PhpParser\Node\Expr\ArrayDimFetch;
@@ -20,6 +19,7 @@
 use PhpParser\Node\Stmt\Trait_;
 use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\ClassReflection;
+use PHPStan\Type\StaticType;
 use PHPStan\Type\TypeWithClassName;
 use Rector\Core\NodeAnalyzer\PropertyFetchAnalyzer;
 use Rector\Core\PhpParser\AstResolver;
@@ -48,8 +48,11 @@ public function __construct(
     /**
      * @return array<PropertyFetch|StaticPropertyFetch>
      */
-    public function findPrivatePropertyFetches(Class_ $class, Property | Param $propertyOrPromotedParam, Scope $scope): array
-    {
+    public function findPrivatePropertyFetches(
+        Class_ $class,
+        Property | Param $propertyOrPromotedParam,
+        Scope $scope
+    ): array {
         $propertyName = $this->resolvePropertyName($propertyOrPromotedParam);
         if ($propertyName === null) {
             return [];
@@ -199,11 +202,11 @@ private function decoratePropertyFetch(Node $node, Scope $scope): void
         }
 
         foreach ($node->getArgs() as $key => $arg) {
-            if (!$arg->value instanceof PropertyFetch && !$arg->value instanceof StaticPropertyFetch) {
+            if (! $arg->value instanceof PropertyFetch && ! $arg->value instanceof StaticPropertyFetch) {
                 continue;
             }
 
-            if (!$this->isFoundByRefParam($node, $key, $scope)) {
+            if (! $this->isFoundByRefParam($node, $key, $scope)) {
                 continue;
             }
 

diff --git a/tests/Issues/SimplifyEmpty/config/configured_rule.php b/tests/Issues/SimplifyEmpty/config/configured_rule.php
@@ -8,7 +8,6 @@
 use Rector\Strict\Rector\Empty_\DisallowedEmptyRuleFixerRector;
 
 return static function (RectorConfig $rectorConfig): void {
-
     $rectorConfig->rules([
         SimplifyIfReturnBoolRector::class,
         SimplifyDeMorganBinaryRector::class,