use rule identifiers

Author: TomasVotruba

README.md

@@ -1028,6 +1028,28 @@ final class SomeClass
 
 <br>
 
+## Doctrine-specific Rules
+
+### NoGetRepositoryOutsideServiceRule
+
+Instead of getting repository from EntityManager, use constructor injection and service pattern to keep code clean
+
+```yaml
+rules:
+    - Symplify\PHPStanRules\Rules\Doctrine\NoGetRepositoryOutsideServiceRule
+```
+
+```php
+class SomeClass
+{
+    public function run(EntityManagerInterface $entityManager)
+    {
+        return $entityManager->getRepository(SomeEntity::class);
+    }
+}
+```
+
+
 <!-- ruledoc-end -->
 
 <br>

composer-dependency-analyser.php

@@ -8,7 +8,6 @@
 return (new Configuration())->addPathToScan(__DIR__ . '/src', false)
     ->addPathToExclude(__DIR__ . '/tests/Rules/Rector/NoInstanceOfStaticReflectionRule/Fixture')
     ->addPathToExclude(__DIR__ . '/tests/Rules/Enum/RequireUniqueEnumConstantRule/Fixture')
-    ->addPathToExclude(__DIR__ . '/tests/Rules/NoReturnArrayVariableListRule/Fixture')
     ->addPathToExclude(__DIR__ . '/tests/Rules/ForbiddenExtendOfNonAbstractClassRule/Fixture')
     ->addPathToExclude(__DIR__ . '/tests/Rules/PHPUnit/NoTestMocksRule/Fixture')
 

config/services/services.neon

@@ -13,3 +13,6 @@ services:
     - Symplify\PHPStanRules\PhpDoc\BarePhpDocParser
     - Symplify\PHPStanRules\PhpDoc\PhpDocResolver
     - Symplify\PHPStanRules\TypeAnalyzer\CallableTypeAnalyzer
+
+    # doctrine
+    - Symplify\PHPStanRules\Doctrine\DoctrineEntityDocumentAnalyser

src/Doctrine/DoctrineEntityDocumentAnalyser.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Symplify\PHPStanRules\PHPStan;
+namespace Symplify\PHPStanRules\Doctrine;
 
 use PHPStan\PhpDoc\ResolvedPhpDocBlock;
 use PHPStan\Reflection\ClassReflection;

src/Enum/ClassName.php

@@ -24,7 +24,7 @@ final class ClassName
     /**
      * @var string
      */
-    public const EVENT_DISPATCHER_INTERFACE = 'Symfony\Component\EventDispatcher\EventSubscriberInterface';
+    public const EVENT_DISPATCHER_INTERFACE = 'Symfony\Component\EventDispatcher\EventDispatcherInterface';
 
     /**
      * @var string

src/Enum/RuleIdentifier.php

@@ -177,4 +177,26 @@ final class RuleIdentifier
     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';
 }

src/PHPStan/Rules/Doctrine/NoGetRepositoryOutsideServiceRule.php

@@ -9,14 +9,11 @@
 use PhpParser\Node\Identifier;
 use PHPStan\Analyser\Scope;
 use PHPStan\Rules\Rule;
-use PHPStan\Rules\RuleError;
 use PHPStan\Rules\RuleErrorBuilder;
+use Symplify\PHPStanRules\Enum\RuleIdentifier;
 
 /**
- * Check if abstract controller has constructor, as it should use
- * #[Require] instead to avoid parent constructor override
- *
- * @see \Symplify\PHPStanRules\Tests\PHPStan\Rule\NoGetRepositoryOutsideServiceRule\NoGetRepositoryOutsideServiceRuleTest
+ * @see \Symplify\PHPStanRules\Tests\Rules\Doctrine\NoGetRepositoryOutsideServiceRule\NoGetRepositoryOutsideServiceRuleTest
  *
  * @implements Rule<MethodCall>
  */
@@ -34,7 +31,6 @@ public function getNodeType(): string
 
     /**
      * @param MethodCall $node
-     * @return RuleError[]
      */
     public function processNode(Node $node, Scope $scope): array
     {
@@ -47,7 +43,10 @@ public function processNode(Node $node, Scope $scope): array
         }
 
         if (! $scope->isInClass()) {
-            $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)->build();
+            $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
+                ->identifier(RuleIdentifier::DOCTRINE_NO_GET_REPOSITORY_OUTSIDE_SERVICE)
+                ->build();
+
             return [$ruleError];
         }
 
@@ -57,7 +56,10 @@ public function processNode(Node $node, Scope $scope): array
             return [];
         }
 
-        $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)->build();
+        $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
+            ->identifier(RuleIdentifier::DOCTRINE_NO_GET_REPOSITORY_OUTSIDE_SERVICE)
+            ->build();
+
         return [$ruleError];
     }
 }

src/PHPStan/Rules/Doctrine/NoParentRepositoryRule.php

@@ -9,8 +9,8 @@
 use PhpParser\Node\Stmt\Class_;
 use PHPStan\Analyser\Scope;
 use PHPStan\Rules\Rule;
-use PHPStan\Rules\RuleError;
 use PHPStan\Rules\RuleErrorBuilder;
+use Symplify\PHPStanRules\Enum\RuleIdentifier;
 
 /**
  * Check if class extends repository class,
@@ -39,7 +39,6 @@ public function getNodeType(): string
 
     /**
      * @param Class_ $node
-     * @return RuleError[]
      */
     public function processNode(Node $node, Scope $scope): array
     {
@@ -52,9 +51,10 @@ public function processNode(Node $node, Scope $scope): array
             return [];
         }
 
-        $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
+        $identifierRuleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
+            ->identifier(RuleIdentifier::DOCTRINE_NO_PARENT_REPOSITORY)
             ->build();
 
-        return [$ruleError];
+        return [$identifierRuleError];
     }
 }

src/PHPStan/Rules/Doctrine/NoRepositoryCallInDataFixtureRule.php

@@ -55,11 +55,11 @@ public function processNode(Node $node, Scope $scope): array
             return [];
         }
 
-        $ruleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
+        $identifierRuleError = RuleErrorBuilder::message(self::ERROR_MESSAGE)
             ->identifier(RuleIdentifier::DOCTRINE_NO_REPOSITORY_CALL_IN_DATA_FIXTURES)
             ->build();
 
-        return [$ruleError];
+        return [$identifierRuleError];
     }
 
     private function isDataFixtureClass(Scope $scope): bool

…c/Testing/DataProviderMethodResolver.php …c/PHPUnit/DataProviderMethodResolver.phpsrc/Testing/DataProviderMethodResolver.php renamed to src/PHPUnit/DataProviderMethodResolver.php src/Testing/DataProviderMethodResolver.php renamed to src/PHPUnit/DataProviderMethodResolver.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Symplify\PHPStanRules\PHPStan;
+namespace Symplify\PHPStanRules\PHPUnit;
 
 use PhpParser\Comment\Doc;
 use PhpParser\Node\Stmt\ClassMethod;