skip dynamic class const fetch (#224)

Author: TomasVotruba

src/Rules/Doctrine/NoGetRepositoryOutsideServiceRule.php

@@ -7,6 +7,7 @@
 use PhpParser\Node;
 use PhpParser\Node\Expr\ClassConstFetch;
 use PhpParser\Node\Expr\MethodCall;
+use PhpParser\Node\Name;
 use PhpParser\Node\Scalar\String_;
 use PHPStan\Analyser\Scope;
 use PHPStan\Rules\Rule;
@@ -76,6 +77,11 @@ private function isDynamicArg(MethodCall $methodCall): bool
             return false;
         }
 
-        return ! $firstArg->value instanceof ClassConstFetch;
+        if ($firstArg->value instanceof ClassConstFetch) {
+            $classConstFetch = $firstArg->value;
+            return ! $classConstFetch->class instanceof Name;
+        }
+
+        return true;
     }
 }

tests/Rules/Doctrine/NoGetRepositoryOutsideServiceRule/Fixture/SkipDynamicClassConstFetch.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Symplify\PHPStanRules\Tests\Rules\Doctrine\NoGetRepositoryOutsideServiceRule\Fixture;
+
+use Doctrine\ORM\EntityManager;
+use Symplify\PHPStanRules\Tests\Rules\Doctrine\NoGetRepositoryOutsideServiceRule\Source\SomeRandomEntity;
+
+final readonly class SkipDynamicClassConstFetch
+{
+    public function run(
+        EntityManager $entityManager,
+        object $someObject
+    ) {
+        $someRepository = $entityManager->getRepository($someObject::class);
+    }
+}

tests/Rules/Doctrine/NoGetRepositoryOutsideServiceRule/Fixture/SkipDymamicFetch.php renamed to tests/Rules/Doctrine/NoGetRepositoryOutsideServiceRule/Fixture/SkipDynamicFetch.php

@@ -7,7 +7,7 @@
 use Doctrine\ORM\EntityManager;
 use Symplify\PHPStanRules\Tests\Rules\Doctrine\NoGetRepositoryOutsideServiceRule\Source\SomeRandomEntity;
 
-final readonly class SkipDymamicFetch
+final readonly class SkipDynamicFetch
 {
     public function run(
         EntityManager $entityManager,

tests/Rules/Doctrine/NoGetRepositoryOutsideServiceRule/NoGetRepositoryOutsideServiceRuleTest.php

@@ -34,7 +34,8 @@ public static function provideData(): Iterator
         ]]];
 
         yield [__DIR__ . '/Fixture/SkipInRepository.php', []];
-        yield [__DIR__ . '/Fixture/SkipDymamicFetch.php', []];
+        yield [__DIR__ . '/Fixture/SkipDynamicFetch.php', []];
+        yield [__DIR__ . '/Fixture/SkipDynamicClassConstFetch.php', []];
     }
 
     protected function getRule(): Rule