skip non-closure sets in NoServiceSameNameSetClassRule

Author: TomasVotruba

src/Rules/Symfony/ConfigClosure/NoServiceSameNameSetClassRule.php

@@ -7,16 +7,19 @@
 use Nette\Utils\Strings;
 use PhpParser\Node;
 use PhpParser\Node\Expr\ClassConstFetch;
+use PhpParser\Node\Expr\Closure;
 use PhpParser\Node\Expr\MethodCall;
+use PhpParser\NodeFinder;
 use PHPStan\Analyser\Scope;
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleError;
 use PHPStan\Rules\RuleErrorBuilder;
 use Symplify\PHPStanRules\Enum\RuleIdentifier\SymfonyRuleIdentifier;
 use Symplify\PHPStanRules\Helper\NamingHelper;
+use Symplify\PHPStanRules\Symfony\NodeAnalyzer\SymfonyClosureDetector;
 
 /**
- * @implements Rule<MethodCall>
+ * @implements Rule<Closure>
  *
  * @see \Symplify\PHPStanRules\Tests\Rules\Symfony\ConfigClosure\NoServiceSameNameSetClassRule\NoServiceSameNameSetClassRuleTest
  */
@@ -27,62 +30,93 @@ final class NoServiceSameNameSetClassRule implements Rule
      */
     public const ERROR_MESSAGE = 'No need to duplicate service class and name. Use only "$services->set(%s::class)" instead';
 
+    private NodeFinder $nodeFinder;
+
+    public function __construct()
+    {
+        $this->nodeFinder = new NodeFinder();
+    }
+
     public function getNodeType(): string
     {
-        return MethodCall::class;
+        return Closure::class;
     }
 
     /**
-     * @param MethodCall $node
+     * @param Closure $node
      * @return RuleError[]
      */
     public function processNode(Node $node, Scope $scope): array
     {
-        if ($node->isFirstClassCallable()) {
+        if (! SymfonyClosureDetector::detect($node)) {
             return [];
         }
 
-        if (! NamingHelper::isName($node->name, 'set')) {
-            return [];
-        }
+        /** @var MethodCall[] $methodCalls */
+        $methodCalls = $this->nodeFinder->findInstanceOf($node, MethodCall::class);
 
-        if (count($node->getArgs()) !== 2) {
-            return [];
+        $ruleErrors = [];
+
+        foreach ($methodCalls as $methodCall) {
+            if ($methodCall->isFirstClassCallable()) {
+                continue;
+            }
+
+            if (! NamingHelper::isName($methodCall->name, 'set')) {
+                continue;
+            }
+
+            if (count($methodCall->getArgs()) !== 2) {
+                continue;
+            }
+
+            $serviceNameValue = $this->matchTwoArgsOfSameClassConstName($methodCall);
+            if (! is_string($serviceNameValue)) {
+                continue;
+            }
+
+            if (str_contains($serviceNameValue, '\\')) {
+                $serviceNameValue = Strings::after($serviceNameValue, '\\', -1);
+            }
+
+            $identifierRuleError = RuleErrorBuilder::message(sprintf(self::ERROR_MESSAGE, $serviceNameValue))
+                ->identifier(SymfonyRuleIdentifier::NO_SERVICE_SAME_NAME_SET_CLASS)
+                ->line($methodCall->getStartLine())
+                ->build();
+
+            $ruleErrors[] = $identifierRuleError;
         }
 
-        $serviceName = $node->getArgs()[0]->value;
-        $serviceType = $node->getArgs()[1]->value;
+        return $ruleErrors;
+    }
+
+    private function matchTwoArgsOfSameClassConstName(MethodCall $methodCall): ?string
+    {
+        $serviceName = $methodCall->getArgs()[0]->value;
+        $serviceType = $methodCall->getArgs()[1]->value;
 
         if (! $serviceName instanceof ClassConstFetch) {
-            return [];
+            return null;
         }
 
         if (! $serviceType instanceof ClassConstFetch) {
-            return [];
+            return null;
         }
 
         $serviceNameValue = NamingHelper::getName($serviceName->class);
         if (! is_string($serviceNameValue)) {
-            return [];
+            return null;
         }
 
         $serviceTypeValue = NamingHelper::getName($serviceType->class);
         if (! is_string($serviceTypeValue)) {
-            return [];
+            return null;
         }
 
         if ($serviceNameValue !== $serviceTypeValue) {
-            return [];
+            return null;
         }
 
-        if (str_contains($serviceNameValue, '\\')) {
-            $serviceNameValue = Strings::after($serviceNameValue, '\\', -1);
-        }
-
-        $identifierRuleError = RuleErrorBuilder::message(sprintf(self::ERROR_MESSAGE, $serviceNameValue))
-            ->identifier(SymfonyRuleIdentifier::NO_SERVICE_SAME_NAME_SET_CLASS)
-            ->build();
-
-        return [$identifierRuleError];
+        return $serviceNameValue;
     }
 }

tests/Rules/Symfony/ConfigClosure/NoServiceSameNameSetClassRule/Fixture/SkipNonClosureConstantSet.php

@@ -0,0 +1,18 @@
+<?php
+
+namespace Symplify\PHPStanRules\Tests\Rules\Symfony\ConfigClosure\NoServiceSameNameSetClassRule\Fixture;
+
+use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+use Symplify\PHPStanRules\Tests\Rules\Symfony\ConfigClosure\NoServiceSameNameSetClassRule\Source\SomeSetService;
+
+class SkipNonClosureConstantSet
+{
+    private const NAME = 'name';
+
+    private const TYPE = 'type';
+
+    public function run()
+    {
+        $this->set(self::NAME, self::TYPE);
+    }
+}

tests/Rules/Symfony/ConfigClosure/NoServiceSameNameSetClassRule/Fixture/SomeConfigWithInvalidSet.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Symplify\PHPStanRules\Tests\Rules\Symfony\ConfigClosure\ServicesExcludedDirectoryMustExistRule\Fixture;
+namespace Symplify\PHPStanRules\Tests\Rules\Symfony\ConfigClosure\NoServiceSameNameSetClassRule\Fixture;
 
 use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
 use Symplify\PHPStanRules\Tests\Rules\Symfony\ConfigClosure\NoServiceSameNameSetClassRule\Source\SomeSetService;

tests/Rules/Symfony/ConfigClosure/NoServiceSameNameSetClassRule/NoServiceSameNameSetClassRuleTest.php

@@ -29,10 +29,20 @@ public static function provideData(): Iterator
                 11,
             ],
         ]];
+
+        yield [__DIR__ . '/Fixture/SkipNonClosureConstantSet.php', []];
+    }
+
+    /**
+     * @return string[]
+     */
+    public static function getAdditionalConfigFiles(): array
+    {
+        return [__DIR__ . '/config/configured_rule.neon'];
     }
 
     protected function getRule(): Rule
     {
-        return new NoServiceSameNameSetClassRule();
+        return self::getContainer()->getByType(NoServiceSameNameSetClassRule::class);
     }
 }

tests/Rules/Symfony/ConfigClosure/NoServiceSameNameSetClassRule/config/configured_rule.neon

@@ -0,0 +1,2 @@
+rules:
+    - Symplify\PHPStanRules\Rules\Symfony\ConfigClosure\NoServiceSameNameSetClassRule