split identifiers by group to easily search through them

TomasVotruba · TomasVotruba · commit 0b2f3c753b58 · 2025-01-07T09:56:36.000Z
diff --git a/src/Enum/DoctrineRuleIdentifier.php b/src/Enum/DoctrineRuleIdentifier.php
@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Symplify\PHPStanRules\Enum;
+
+final class DoctrineRuleIdentifier
+{
+    public const DOCTRINE_NO_GET_REPOSITORY_OUTSIDE_SERVICE = 'doctrine.noGetRepositoryOutsideService';
+
+    public const DOCTRINE_NO_REPOSITORY_CALL_IN_DATA_FIXTURES = 'doctrine.noRepositoryCallInDataFixtures';
+
+    public const DOCTRINE_NO_PARENT_REPOSITORY = 'doctrine.noParentRepository';
+
+    public const NO_ENTITY_MOCKING = 'doctrine.noEntityMocking';
+
+    public const REQUIRE_QUERY_BUILDER_ON_REPOSITORY = 'doctrine.requireQueryBuilderOnRepository';
+}
diff --git a/src/Enum/PHPUnitRuleIdentifier.php b/src/Enum/PHPUnitRuleIdentifier.php
@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Symplify\PHPStanRules\Enum;
+
+final class PHPUnitRuleIdentifier
+{
+    public const NO_DOCUMENT_MOCKING = 'phpunit.noDocumentMocking';
+
+    public const NO_MOCK_ONLY = 'phpunit.noMockOnly';
+
+    public const PUBLIC_STATIC_DATA_PROVIDER = 'phpunit.publicStaticDataProvider';
+}
diff --git a/src/Enum/RectorRuleIdentifier.php b/src/Enum/RectorRuleIdentifier.php
@@ -0,0 +1,16 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Symplify\PHPStanRules\Enum;
+
+final class RectorRuleIdentifier
+{
+    public const NO_INSTANCE_OF_STATIC_REFLECTION = 'rector.noInstanceOfStaticReflection';
+
+    public const UPGRADE_DOWNGRADE_REGISTERED_IN_SET = 'rector.upgradeDowngradeRegisteredInSet';
+
+    public const PHP_RULE_IMPLEMENTS_MIN_VERSION = 'rector.phpRuleImplementsMinVersion';
+
+    public const NO_CLASS_REFLECTION_STATIC_REFLECTION = 'rector.noClassReflectionStaticReflection';
+}
diff --git a/src/Enum/RuleIdentifier.php b/src/Enum/RuleIdentifier.php
@@ -12,16 +12,8 @@ final class RuleIdentifier
 
     public const REQUIRE_ATTRIBUTE_NAME = 'symplify.requireAttributeName';
 
-    public const RECTOR_PHP_RULE_IMPLEMENTS_MIN_VERSION = 'rector.phpRuleImplementsMinVersion';
-
-    public const RECTOR_UPGRADE_DOWNGRADE_REGISTERED_IN_SET = 'rector.upgradeDowngradeRegisteredInSet';
-
     public const PHP_PARSER_NO_LEADING_BACKSLASH_IN_NAME = 'phpParser.noLeadingBackslashInName';
 
-    public const RECTOR_NO_INSTANCE_OF_STATIC_REFLECTION = 'rector.noInstanceOfStaticReflection';
-
-    public const RECTOR_NO_CLASS_REFLECTION_STATIC_REFLECTION = 'rector.noClassReflectionStaticReflection';
-
     public const PARENT_METHOD_VISIBILITY_OVERRIDE = 'symplify.parentMethodVisibilityOverride';
 
     public const NO_RETURN_SETTER_METHOD = 'symplify.noReturnSetterMethod';
@@ -62,47 +54,15 @@ final class RuleIdentifier
 
     public const REQUIRED_INTERFACE_CONTRACT_NAMESPACE = 'symplify.requiredInterfaceContractNamespace';
 
-    public const SYMFONY_REQUIRE_INVOKABLE_CONTROLLER = 'symfony.requireInvokableController';
-
     public const NO_VALUE_OBJECT_IN_SERVICE_CONSTRUCTOR = 'symplify.noValueObjectInServiceConstructor';
 
-    public const DOCTRINE_NO_REPOSITORY_CALL_IN_DATA_FIXTURES = 'doctrine.noRepositoryCallInDataFixtures';
-
-    public const PHPUNIT_NO_DOCUMENT_MOCKING = 'phpunit.noDocumentMocking';
-
     public const NO_DYNAMIC_NAME = 'symplify.noDynamicName';
 
     public const NO_REFERENCE = 'symplify.noReference';
 
-    public const PHPUNIT_NO_MOCK_ONLY = 'phpunit.noMockOnly';
-
-    public const SINGLE_ARG_EVENT_DISPATCH = 'symfony.singleArgEventDispatch';
-
-    public const NO_ENTITY_MOCKING = 'doctrine.noEntityMocking';
-
-    public const NO_STRING_IN_GET_SUBSCRIBED_EVENTS = 'symfony.noStringInGetSubscribedEvents';
-
-    public const NO_LISTENER_WITHOUT_CONTRACT = 'symfony.noListenerWithoutContract';
-
-    public const DOCTRINE_NO_PARENT_REPOSITORY = 'doctrine.noParentRepository';
-
-    public const DOCTRINE_NO_GET_REPOSITORY_OUTSIDE_SERVICE = 'doctrine.noGetRepositoryOutsideService';
-
-    public const SYMFONY_NO_REQUIRED_OUTSIDE_CLASS = 'symfony.noRequiredOutsideClass';
-
     public const NO_CONSTRUCTOR_OVERRIDE = 'symplify.noConstructorOverride';
 
-    public const SYMFONY_NO_ABSTRACT_CONTROLLER_CONSTRUCTOR = 'symfony.noAbstractControllerConstructor';
-
-    public const PHPUNIT_PUBLIC_STATIC_DATA_PROVIDER = 'phpunit.publicStaticDataProvider';
-
     public const FORBIDDEN_NEW_INSTANCE = 'symplify.forbiddenNewInstance';
 
-    public const REQUIRE_QUERY_BUILDER_ON_REPOSITORY = 'doctrine.requireQueryBuilderOnRepository';
-
-    public const NO_GET_IN_CONTROLLER = 'symfony.noGetInController';
-
-    public const NO_GET_DOCTRINE_IN_CONTROLLER = 'symfony.noGetDoctrineInController';
-
     public const MAXIMUM_IGNORED_ERROR_COUNT = 'symplify.maximumIgnoredErrorCount';
 }
diff --git a/src/Enum/SymfonyRuleIdentifier.php b/src/Enum/SymfonyRuleIdentifier.php
@@ -0,0 +1,26 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Symplify\PHPStanRules\Enum;
+
+final class SymfonyRuleIdentifier
+{
+    public const NO_GET_IN_CONTROLLER = 'symfony.noGetInController';
+
+    public const NO_GET_DOCTRINE_IN_CONTROLLER = 'symfony.noGetDoctrineInController';
+
+    public const SINGLE_ARG_EVENT_DISPATCH = 'symfony.singleArgEventDispatch';
+
+    public const NO_LISTENER_WITHOUT_CONTRACT = 'symfony.noListenerWithoutContract';
+
+    public const SYMFONY_REQUIRE_INVOKABLE_CONTROLLER = 'symfony.requireInvokableController';
+
+    public const SYMFONY_NO_REQUIRED_OUTSIDE_CLASS = 'symfony.noRequiredOutsideClass';
+
+    public const NO_STRING_IN_GET_SUBSCRIBED_EVENTS = 'symfony.noStringInGetSubscribedEvents';
+
+    public const SYMFONY_NO_ABSTRACT_CONTROLLER_CONSTRUCTOR = 'symfony.noAbstractControllerConstructor';
+
+    public const SINGLE_REQUIRED_METHOD = 'symfony.singleRequiredMethod';
+}
diff --git a/src/Rules/Doctrine/NoDocumentMockingRule.php b/src/Rules/Doctrine/NoDocumentMockingRule.php
@@ -10,7 +10,7 @@
 use PHPStan\Analyser\Scope;
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\PHPUnitRuleIdentifier;
 
 /**
  * @implements Rule<MethodCall>
@@ -53,7 +53,7 @@ public function processNode(Node $node, Scope $scope): array
             }
 
             $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
-                ->identifier(RuleIdentifier::PHPUNIT_NO_DOCUMENT_MOCKING)
+                ->identifier(PHPUnitRuleIdentifier::NO_DOCUMENT_MOCKING)
                 ->build();
 
             return [$ruleError];
diff --git a/src/Rules/Doctrine/NoEntityMockingRule.php b/src/Rules/Doctrine/NoEntityMockingRule.php
@@ -11,7 +11,7 @@
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
 use Symplify\PHPStanRules\Doctrine\DoctrineEntityDocumentAnalyser;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\DoctrineRuleIdentifier;
 use Symplify\PHPStanRules\NodeAnalyzer\MethodCallNameAnalyzer;
 
 /**
@@ -62,7 +62,7 @@ public function processNode(Node $node, Scope $scope): array
             }
 
             $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
-                ->identifier(RuleIdentifier::NO_ENTITY_MOCKING)
+                ->identifier(DoctrineRuleIdentifier::NO_ENTITY_MOCKING)
                 ->build();
 
             return [$ruleError];
diff --git a/src/Rules/Doctrine/NoGetRepositoryOutsideServiceRule.php b/src/Rules/Doctrine/NoGetRepositoryOutsideServiceRule.php
@@ -10,7 +10,7 @@
 use PHPStan\Analyser\Scope;
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\DoctrineRuleIdentifier;
 
 /**
  * @see \Symplify\PHPStanRules\Tests\Rules\Doctrine\NoGetRepositoryOutsideServiceRule\NoGetRepositoryOutsideServiceRuleTest
@@ -44,7 +44,7 @@ public function processNode(Node $node, Scope $scope): array
 
         if (! $scope->isInClass()) {
             $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
-                ->identifier(RuleIdentifier::DOCTRINE_NO_GET_REPOSITORY_OUTSIDE_SERVICE)
+                ->identifier(DoctrineRuleIdentifier::DOCTRINE_NO_GET_REPOSITORY_OUTSIDE_SERVICE)
                 ->build();
 
             return [$ruleError];
@@ -57,7 +57,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
-            ->identifier(RuleIdentifier::DOCTRINE_NO_GET_REPOSITORY_OUTSIDE_SERVICE)
+            ->identifier(DoctrineRuleIdentifier::DOCTRINE_NO_GET_REPOSITORY_OUTSIDE_SERVICE)
             ->build();
 
         return [$ruleError];
diff --git a/src/Rules/Doctrine/NoParentRepositoryRule.php b/src/Rules/Doctrine/NoParentRepositoryRule.php
@@ -11,7 +11,7 @@
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
 use Symplify\PHPStanRules\Enum\ClassName;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\DoctrineRuleIdentifier;
 
 /**
  * Check if class extends repository class,
@@ -48,7 +48,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         $identifierRuleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
-            ->identifier(RuleIdentifier::DOCTRINE_NO_PARENT_REPOSITORY)
+            ->identifier(DoctrineRuleIdentifier::DOCTRINE_NO_PARENT_REPOSITORY)
             ->build();
 
         return [$identifierRuleError];
diff --git a/src/Rules/Doctrine/NoRepositoryCallInDataFixtureRule.php b/src/Rules/Doctrine/NoRepositoryCallInDataFixtureRule.php
@@ -12,7 +12,7 @@
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
 use Symplify\PHPStanRules\Enum\ClassName;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\DoctrineRuleIdentifier;
 use Symplify\PHPStanRules\Tests\Rules\Doctrine\NoRepositoryCallInDataFixtureRule\NoRepositoryCallInDataFixtureRuleTest;
 
 /**
@@ -56,7 +56,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         $identifierRuleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
-            ->identifier(RuleIdentifier::DOCTRINE_NO_REPOSITORY_CALL_IN_DATA_FIXTURES)
+            ->identifier(DoctrineRuleIdentifier::DOCTRINE_NO_REPOSITORY_CALL_IN_DATA_FIXTURES)
             ->build();
 
         return [$identifierRuleError];
diff --git a/src/Rules/Doctrine/RequireQueryBuilderOnRepositoryRule.php b/src/Rules/Doctrine/RequireQueryBuilderOnRepositoryRule.php
@@ -12,7 +12,7 @@
 use PHPStan\Rules\RuleErrorBuilder;
 use PHPStan\Type\ObjectType;
 use Symplify\PHPStanRules\Enum\ClassName;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\DoctrineRuleIdentifier;
 
 /**
  * @implements Rule<MethodCall>
@@ -53,7 +53,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         $identifierRuleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
-            ->identifier(RuleIdentifier::REQUIRE_QUERY_BUILDER_ON_REPOSITORY)
+            ->identifier(DoctrineRuleIdentifier::REQUIRE_QUERY_BUILDER_ON_REPOSITORY)
             ->build();
 
         return [$identifierRuleError];
diff --git a/src/Rules/PHPUnit/NoMockOnlyTestRule.php b/src/Rules/PHPUnit/NoMockOnlyTestRule.php
@@ -12,7 +12,7 @@
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
 use Symplify\PHPStanRules\Enum\ClassName;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\PHPUnitRuleIdentifier;
 use Symplify\PHPStanRules\Testing\PHPUnitTestAnalyser;
 
 /**
@@ -69,7 +69,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         $identifierRuleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
-            ->identifier(RuleIdentifier::PHPUNIT_NO_MOCK_ONLY)
+            ->identifier(PHPUnitRuleIdentifier::NO_MOCK_ONLY)
             ->build();
 
         return [$identifierRuleError];
diff --git a/src/Rules/PHPUnit/PublicStaticDataProviderRule.php b/src/Rules/PHPUnit/PublicStaticDataProviderRule.php
@@ -10,7 +10,7 @@
 use PHPStan\Node\InClassNode;
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\PHPUnitRuleIdentifier;
 use Symplify\PHPStanRules\PHPUnit\DataProviderMethodResolver;
 use Symplify\PHPStanRules\Testing\PHPUnitTestAnalyser;
 
@@ -76,7 +76,7 @@ public function processNode(Node $node, Scope $scope): array
             if (! $dataProviderClassMethod->isPublic()) {
                 $errorMessage = sprintf(self::PUBLIC_ERROR_MESSAGE, $dataProviderMethodName);
                 $ruleErrors[] = RuleErrorBuilder::message($errorMessage)
-                    ->identifier(RuleIdentifier::PHPUNIT_PUBLIC_STATIC_DATA_PROVIDER)
+                    ->identifier(PHPUnitRuleIdentifier::PUBLIC_STATIC_DATA_PROVIDER)
                     ->line($dataProviderClassMethod->getLine())
                     ->build();
             }
diff --git a/src/Rules/Rector/NoClassReflectionStaticReflectionRule.php b/src/Rules/Rector/NoClassReflectionStaticReflectionRule.php
@@ -11,7 +11,7 @@
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
 use ReflectionClass;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\RectorRuleIdentifier;
 use Symplify\PHPStanRules\TypeAnalyzer\RectorAllowedAutoloadedTypeAnalyzer;
 
 /**
@@ -56,7 +56,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         return [RuleErrorBuilder::message(self::ERROR_MESSAGE)
-            ->identifier(RuleIdentifier::RECTOR_NO_CLASS_REFLECTION_STATIC_REFLECTION)
+            ->identifier(RectorRuleIdentifier::NO_CLASS_REFLECTION_STATIC_REFLECTION)
             ->build()];
     }
 }
diff --git a/src/Rules/Rector/NoInstanceOfStaticReflectionRule.php b/src/Rules/Rector/NoInstanceOfStaticReflectionRule.php
@@ -14,7 +14,7 @@
 use PHPStan\Rules\RuleErrorBuilder;
 use PHPStan\Type\Constant\ConstantStringType;
 use PHPStan\Type\Type;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\RectorRuleIdentifier;
 use Symplify\PHPStanRules\TypeAnalyzer\RectorAllowedAutoloadedTypeAnalyzer;
 
 /**
@@ -55,7 +55,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         return [RuleErrorBuilder::message(self::ERROR_MESSAGE)
-            ->identifier(RuleIdentifier::RECTOR_NO_INSTANCE_OF_STATIC_REFLECTION)
+            ->identifier(RectorRuleIdentifier::NO_INSTANCE_OF_STATIC_REFLECTION)
             ->build()];
     }
 
diff --git a/src/Rules/Rector/PhpUpgradeDowngradeRegisteredInSetRule.php b/src/Rules/Rector/PhpUpgradeDowngradeRegisteredInSetRule.php
@@ -15,7 +15,7 @@
 use Rector\Set\ValueObject\SetList;
 use SplFileInfo;
 use Symplify\PHPStanRules\Enum\ClassName;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\RectorRuleIdentifier;
 use Symplify\PHPStanRules\Exception\ShouldNotHappenException;
 use Symplify\PHPStanRules\FileSystem\FileSystem;
 
@@ -66,7 +66,7 @@ public function processNode(Node $node, Scope $scope): array
 
         $errorMessage = $this->createErrorMessage($configFilePath, $className);
         return [RuleErrorBuilder::message($errorMessage)
-            ->identifier(RuleIdentifier::RECTOR_UPGRADE_DOWNGRADE_REGISTERED_IN_SET)
+            ->identifier(RectorRuleIdentifier::UPGRADE_DOWNGRADE_REGISTERED_IN_SET)
             ->build()];
     }
 
diff --git a/src/Rules/Rector/PhpUpgradeImplementsMinPhpVersionInterfaceRule.php b/src/Rules/Rector/PhpUpgradeImplementsMinPhpVersionInterfaceRule.php
@@ -12,7 +12,7 @@
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
 use Rector\VersionBonding\Contract\MinPhpVersionInterface;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\RectorRuleIdentifier;
 
 /**
  * @see \Symplify\PHPStanRules\Tests\Rules\Rector\PhpUpgradeImplementsMinPhpVersionInterfaceRule\PhpUpgradeImplementsMinPhpVersionInterfaceRuleTest
@@ -66,7 +66,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         $identifierRuleError = RuleErrorBuilder::message(sprintf(self::ERROR_MESSAGE, $className))
-            ->identifier(RuleIdentifier::RECTOR_PHP_RULE_IMPLEMENTS_MIN_VERSION)
+            ->identifier(RectorRuleIdentifier::PHP_RULE_IMPLEMENTS_MIN_VERSION)
             ->build();
 
         return [$identifierRuleError];
diff --git a/src/Rules/Symfony/NoAbstractControllerConstructorRule.php b/src/Rules/Symfony/NoAbstractControllerConstructorRule.php
@@ -10,7 +10,7 @@
 use PHPStan\Analyser\Scope;
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
-use Symplify\PHPStanRules\Enum\RuleIdentifier;
+use Symplify\PHPStanRules\Enum\SymfonyRuleIdentifier;
 
 /**
  * Check if abstract controller has constructor, as it should use
@@ -55,7 +55,7 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         $identifierRuleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
-            ->identifier(RuleIdentifier::SYMFONY_NO_ABSTRACT_CONTROLLER_CONSTRUCTOR)
+            ->identifier(SymfonyRuleIdentifier::SYMFONY_NO_ABSTRACT_CONTROLLER_CONSTRUCTOR)
             ->build();
 
         return [$identifierRuleError];
diff --git a/src/Rules/Symfony/NoGetDoctrineInControllerRule.php b/src/Rules/Symfony/NoGetDoctrineInControllerRule.php
diff --git a/src/Rules/Symfony/NoGetInControllerRule.php b/src/Rules/Symfony/NoGetInControllerRule.php
diff --git a/src/Rules/Symfony/NoListenerWithoutContractRule.php b/src/Rules/Symfony/NoListenerWithoutContractRule.php
diff --git a/src/Rules/Symfony/NoRequiredOutsideClassRule.php b/src/Rules/Symfony/NoRequiredOutsideClassRule.php
diff --git a/src/Rules/Symfony/NoStringInGetSubscribedEventsRule.php b/src/Rules/Symfony/NoStringInGetSubscribedEventsRule.php
diff --git a/src/Rules/Symfony/RequireInvokableControllerRule.php b/src/Rules/Symfony/RequireInvokableControllerRule.php
diff --git a/src/Rules/Symfony/SingleArgEventDispatchRule.php b/src/Rules/Symfony/SingleArgEventDispatchRule.php
diff --git a/src/Rules/Symfony/SingleRequiredMethodRule.php b/src/Rules/Symfony/SingleRequiredMethodRule.php
