tidy up

Author: TomasVotruba

rector.php

@@ -23,7 +23,10 @@
         '*/Source/*',
         '*/Source*/*',
         '*/tests/*/Fixture*/Expected/*',
-        StringClassNameToClassConstantRector::class => [__DIR__ . '/config', __DIR__ . '/src/Enum'],
+        StringClassNameToClassConstantRector::class => [
+            __DIR__ . '/config', __DIR__ . '/src/Enum',
+            __DIR__ . '/rules/CodeQuality/Enum/',
+        ],
         UseClassKeywordForClassNameResolutionRector::class => [__DIR__ . '/config'],
 
         RenameForeachValueVariableToMatchMethodCallReturnTypeRector::class => [
@@ -34,19 +37,8 @@
         // marked as skipped
         ReturnNeverTypeRector::class => ['*/tests/*'],
     ])
-    ->withConfiguredRule(StringClassNameToClassConstantRector::class, [
-        'Error',
-        'Exception',
-        'Symfony\*',
-        'Twig_*',
-        'Twig*',
-        'Swift_*',
-        'Doctrine\*',
-        // loaded from project itself
-        'Psr\Container\ContainerInterface',
-        'Symfony\Component\Routing\RouterInterface',
-        'Symfony\Component\DependencyInjection\Container',
-    ])
+    // @todo cleanup rest and move to enum classes ass single place for class names
+    ->withConfiguredRule(StringClassNameToClassConstantRector::class, ['Symfony\*', 'Twig_*', 'Twig*'])
     ->withPhpSets()
     ->withPreparedSets(
         deadCode: true,

rules/CodeQuality/Rector/ClassMethod/RemoveUnusedRequestParamRector.php

@@ -17,6 +17,7 @@
 use Rector\Rector\AbstractRector;
 use Rector\Reflection\ReflectionResolver;
 use Rector\Symfony\TypeAnalyzer\ControllerAnalyzer;
+use Symplify\PHPStanRules\Enum\SymfonyClass;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 
@@ -110,7 +111,7 @@ public function refactor(Node $node): ?Node
                     continue;
                 }
 
-                if (! $this->isObjectType($param->type, new ObjectType('Symfony\Component\HttpFoundation\Request'))) {
+                if (! $this->isObjectType($param->type, new ObjectType(SymfonyClass::REQUEST))) {
                     continue;
                 }
 

rules/CodeQuality/Rector/ClassMethod/ResponseReturnTypeControllerActionRector.php

@@ -25,6 +25,7 @@
 use Rector\Symfony\CodeQuality\Enum\ResponseClass;
 use Rector\Symfony\Enum\SensioAttribute;
 use Rector\Symfony\Enum\SymfonyAnnotation;
+use Rector\Symfony\Enum\SymfonyClass;
 use Rector\Symfony\TypeAnalyzer\ControllerAnalyzer;
 use Rector\TypeDeclaration\NodeAnalyzer\ReturnAnalyzer;
 use Rector\TypeDeclaration\TypeInferer\ReturnTypeInferer;
@@ -125,7 +126,8 @@ public function refactor(Node $node): ?Node
 
             $returnType = $this->returnTypeInferer->inferFunctionLike($node);
             $types = $returnType instanceof UnionType ? $returnType->getTypes() : [$returnType];
-            $objectType = new ObjectType('Symfony\Component\HttpFoundation\Response');
+
+            $objectType = new ObjectType(SymfonyClass::RESPONSE);
 
             foreach ($types as $type) {
                 if ($type instanceof ObjectType && $objectType->isSuperTypeOf($type)->yes()) {

rules/Symfony30/Rector/ClassMethod/GetRequestRector.php

@@ -16,6 +16,7 @@
 use Rector\PhpParser\Node\BetterNodeFinder;
 use Rector\Rector\AbstractRector;
 use Rector\Symfony\Bridge\NodeAnalyzer\ControllerMethodAnalyzer;
+use Rector\Symfony\Enum\SymfonyClass;
 use Rector\Symfony\TypeAnalyzer\ControllerAnalyzer;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
@@ -25,11 +26,6 @@
  */
 final class GetRequestRector extends AbstractRector
 {
-    /**
-     * @var string
-     */
-    private const REQUEST_CLASS = 'Symfony\Component\HttpFoundation\Request';
-
     private ?string $requestVariableAndParamName = null;
 
     public function __construct(
@@ -230,7 +226,7 @@ private function refactorClassMethod(ClassMethod $classMethod): null|ClassMethod
             return null;
         }
 
-        $fullyQualified = new FullyQualified(self::REQUEST_CLASS);
+        $fullyQualified = new FullyQualified(SymfonyClass::REQUEST);
         $classMethod->params[] = new Param(new Variable(
             $this->getRequestVariableAndParamName()
         ), null, $fullyQualified);

rules/Symfony51/Rector/ClassMethod/CommandConstantReturnCodeRector.php

@@ -13,6 +13,7 @@
 use Rector\PhpParser\Node\BetterNodeFinder;
 use Rector\Rector\AbstractRector;
 use Rector\Reflection\ReflectionResolver;
+use Rector\Symfony\Enum\SymfonyClass;
 use Rector\Symfony\ValueObject\ConstantMap\SymfonyCommandConstantMap;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
@@ -80,7 +81,7 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        if (! $classReflection->is('Symfony\Component\Console\Command\Command')) {
+        if (! $classReflection->is(SymfonyClass::COMMAND)) {
             return null;
         }
 
@@ -121,7 +122,7 @@ private function convertNumberToConstant(Int_ $int): ?ClassConstFetch
         }
 
         return $this->nodeFactory->createClassConstFetch(
-            'Symfony\Component\Console\Command\Command',
+            SymfonyClass::COMMAND,
             SymfonyCommandConstantMap::RETURN_TO_CONST[$int->value]
         );
     }

rules/Symfony61/Rector/Class_/MagicClosureTwigExtensionToNativeMethodsRector.php

@@ -16,6 +16,7 @@
 use Rector\NodeCollector\ValueObject\ArrayCallable;
 use Rector\PHPStan\ScopeFetcher;
 use Rector\Rector\AbstractRector;
+use Rector\Symfony\Enum\TwigClass;
 use Rector\ValueObject\PhpVersion;
 use Rector\VersionBonding\Contract\MinPhpVersionInterface;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
@@ -100,7 +101,7 @@ public function refactor(Node $node): ?Node
     {
         if (! $this->nodeTypeResolver->isObjectTypes($node, [
             new ObjectType('Twig_ExtensionInterface'),
-            new ObjectType('Twig\Extension\ExtensionInterface'),
+            new ObjectType(TwigClass::EXTENSION_INTERFACE),
         ])) {
             return null;
         }

src/Bridge/Symfony/ContainerServiceProvider.php

@@ -7,6 +7,7 @@
 use Rector\Configuration\Option;
 use Rector\Configuration\Parameter\SimpleParameterProvider;
 use Rector\Exception\ShouldNotHappenException;
+use Rector\Symfony\Enum\SymfonyClass;
 use Symfony\Component\DependencyInjection\Container;
 use Webmozart\Assert\Assert;
 
@@ -37,7 +38,7 @@ private function getSymfonyContainer(): object
             $container = require $symfonyContainerPhp;
 
             // this allows older Symfony versions, e.g. 2.8 did not have the PSR yet
-            Assert::isInstanceOf($container, 'Symfony\Component\DependencyInjection\Container');
+            Assert::isInstanceOf($container, SymfonyClass::CONTAINER);
             $this->container = $container;
         }
 

src/Enum/SymfonyClass.php

@@ -226,4 +226,6 @@ final class SymfonyClass
     public const ROUTING_CONFIGURATOR = 'Symfony\Component\Routing\Loader\Configurator\RoutingConfigurator';
 
     public const KERNEL = 'Symfony\Component\HttpKernel\Kernel';
+
+    public const CONTAINER = 'Symfony\Component\DependencyInjection\Container';
 }

src/Enum/TwigClass.php

@@ -20,4 +20,9 @@ final class TwigClass
      * @var string
      */
     public const AS_TWIG_FUNCTION_ATTRIBUTE = 'Twig\Attribute\AsTwigFunction';
+
+    /**
+     * @var string
+     */
+    public const EXTENSION_INTERFACE = 'Twig\Extension\ExtensionInterface';
 }