cs

Author: TomasVotruba

ecs.php

@@ -5,14 +5,7 @@
 use Symplify\EasyCodingStandard\Config\ECSConfig;
 
 return ECSConfig::configure()
-    ->withPaths([
-        __DIR__ . '/config',
-        __DIR__ . '/src',
-        __DIR__ . '/tests',
-    ])
+    ->withPaths([__DIR__ . '/config', __DIR__ . '/src', __DIR__ . '/tests'])
     ->withRootFiles()
-    ->withPreparedSets(psr12: true, common: true)
-    ->withSkip([
-        '*/Source/*',
-        '*/Fixture/*',
-    ]);
+    ->withPreparedSets(psr12: true, common: true, symplify: true)
+    ->withSkip(['*/Source/*', '*/Fixture/*']);

rector.php

@@ -16,13 +16,6 @@
         naming: true,
         earlyReturn: true
     )
-    ->withPaths([
-        __DIR__ . '/config',
-        __DIR__ . '/src',
-        __DIR__ . '/tests',
-    ])
+    ->withPaths([__DIR__ . '/config', __DIR__ . '/src', __DIR__ . '/tests'])
     ->withImportNames(removeUnusedImports: true)
-    ->withSkip([
-        '*/Source/*',
-        '*/Fixture/*',
-    ]);
+    ->withSkip(['*/Source/*', '*/Fixture/*']);

src/Collector/ClassMethod/PublicClassMethodParamTypesCollector.php

@@ -71,7 +71,10 @@ public function processNode(Node $node, Scope $scope): ?array
             return null;
         }
 
-        $printedParamTypesString = $this->collectorMetadataPrinter->printParamTypesToString($node, $classReflection->getName());
+        $printedParamTypesString = $this->collectorMetadataPrinter->printParamTypesToString(
+            $node,
+            $classReflection->getName()
+        );
         return [$classReflection->getName(), $methodName, $printedParamTypesString, $node->getLine()];
     }
 }

src/Collector/MethodCall/MethodCallArgTypesCollector.php

@@ -59,7 +59,11 @@ public function processNode(Node $node, Scope $scope): ?array
 
         $classMethodReference = $this->createClassMethodReference($classMethodCallReference);
 
-        $stringArgTypesString = $this->collectorMetadataPrinter->printArgTypesAsString($node, $methodReflection, $scope);
+        $stringArgTypesString = $this->collectorMetadataPrinter->printArgTypesAsString(
+            $node,
+            $methodReflection,
+            $scope
+        );
         return [$classMethodReference, $stringArgTypesString];
     }
 

src/Matcher/ClassMethodCallReferenceResolver.php

@@ -14,8 +14,11 @@
 
 final class ClassMethodCallReferenceResolver
 {
-    public function resolve(MethodCall|MethodCallableNode $methodCallOrMethodCallable, Scope $scope, bool $allowThisType): ?MethodCallReference
-    {
+    public function resolve(
+        MethodCall|MethodCallableNode $methodCallOrMethodCallable,
+        Scope $scope,
+        bool $allowThisType
+    ): ?MethodCallReference {
         if ($methodCallOrMethodCallable instanceof MethodCallableNode) {
             $methodName = $methodCallOrMethodCallable->getName();
             $variable = $methodCallOrMethodCallable->getVar();

src/Matcher/Collector/PublicClassMethodMatcher.php

@@ -13,7 +13,10 @@ final class PublicClassMethodMatcher
     /**
      * @var string[]
      */
-    private const SKIPPED_TYPES = ['PHPUnit\Framework\TestCase', 'Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator'];
+    private const SKIPPED_TYPES = [
+        'PHPUnit\Framework\TestCase',
+        'Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator',
+    ];
 
     public function shouldSkipClassReflection(ClassReflection $classReflection): bool
     {

src/Printer/CollectorMetadataPrinter.php

@@ -38,9 +38,17 @@ public function __construct(
         $this->standard = new Standard();
     }
 
-    public function printArgTypesAsString(MethodCall $methodCall, ExtendedMethodReflection $extendedMethodReflection, Scope $scope): string
-    {
-        $parametersAcceptor = ParametersAcceptorSelector::selectFromArgs($scope, $methodCall->getArgs(), $extendedMethodReflection->getVariants(), $extendedMethodReflection->getNamedArgumentsVariants());
+    public function printArgTypesAsString(
+        MethodCall $methodCall,
+        ExtendedMethodReflection $extendedMethodReflection,
+        Scope $scope
+    ): string {
+        $parametersAcceptor = ParametersAcceptorSelector::selectFromArgs(
+            $scope,
+            $methodCall->getArgs(),
+            $extendedMethodReflection->getVariants(),
+            $extendedMethodReflection->getNamedArgumentsVariants()
+        );
         $parameters = $parametersAcceptor->getParameters();
 
         $stringArgTypes = [];
@@ -126,8 +134,10 @@ private function transformSelfToClassName(Node $node, ?string $className): Node
         return new FullyQualified($className);
     }
 
-    private function resolveSortedTypes(UnionType|NodeIntersectionType $paramType, ?string $className): UnionType|NodeIntersectionType
-    {
+    private function resolveSortedTypes(
+        UnionType|NodeIntersectionType $paramType,
+        ?string $className
+    ): UnionType|NodeIntersectionType {
         $typeNames = [];
 
         foreach ($paramType->types as $type) {
@@ -183,7 +193,8 @@ private function printTypeToString(Type $type): string
         }
 
         if (count($type->getEnumCases()) === 1) {
-            return $type->getEnumCases()[0]->getClassName();
+            return $type->getEnumCases()[0]
+                ->getClassName();
         }
 
         return $type->describe(VerbosityLevel::typeOnly());

src/Rules/NarrowReturnObjectTypeRule.php

@@ -137,6 +137,7 @@ private function shouldSkipReturnExprType(Type $type): bool
             return true;
         }
 
-        return $type->getObjectClassReflections()[0]->isAnonymous();
+        return $type->getObjectClassReflections()[0]
+            ->isAnonymous();
     }
 }

src/Rules/NoArrayAccessOnObjectRule.php

@@ -26,7 +26,14 @@ final class NoArrayAccessOnObjectRule implements Rule
     /**
      * @var string[]
      */
-    private const ALLOWED_CLASSES = ['SplFixedArray', 'SimpleXMLElement', 'Iterator', 'Aws\ResultInterface', 'Symfony\Component\Form\FormInterface', 'Symfony\Component\OptionsResolver\Options'];
+    private const ALLOWED_CLASSES = [
+        'SplFixedArray',
+        'SimpleXMLElement',
+        'Iterator',
+        'Aws\ResultInterface',
+        'Symfony\Component\Form\FormInterface',
+        'Symfony\Component\OptionsResolver\Options',
+    ];
 
     /**
      * @return class-string<Node>

tests/Rules/NarrowPublicClassMethodParamTypeRule/NarrowPublicClassMethodParamTypeRuleTest.php

@@ -146,26 +146,16 @@ public static function provideData(): Iterator
             __DIR__ . '/Source/ExpectedThisType/CallByThisFromInterface.php',
         ], [[$argErrorMessage, 11]]];
 
-        yield [[
-            __DIR__ . '/Fixture/SkipSelf.php',
-        ], []];
+        yield [[__DIR__ . '/Fixture/SkipSelf.php'], []];
 
-        yield [[
-            __DIR__ . '/Fixture/SkipClosure.php',
-        ], []];
+        yield [[__DIR__ . '/Fixture/SkipClosure.php'], []];
 
-        yield [[
-            __DIR__ . '/Fixture/SkipCallable.php',
-        ], []];
+        yield [[__DIR__ . '/Fixture/SkipCallable.php'], []];
 
-        yield [[
-            __DIR__ . '/Fixture/SkipEnum.php',
-        ], []];
+        yield [[__DIR__ . '/Fixture/SkipEnum.php'], []];
 
         $argErrorMessage = sprintf(NarrowPublicClassMethodParamTypeRule::ERROR_MESSAGE, 'int');
-        yield [[
-            __DIR__ . '/Fixture/HandleDefaultValue.php',
-        ], [[$argErrorMessage, 15]]];
+        yield [[__DIR__ . '/Fixture/HandleDefaultValue.php'], [[$argErrorMessage, 15]]];
     }
 
     /**