Add SymfonyCmfRouteObjectInterfaceConstantsRule

Author: mglaman

rules.neon

@@ -15,3 +15,4 @@ rules:
 	- mglaman\PHPStanDrupal\Rules\Drupal\ModuleLoadInclude
 	- mglaman\PHPStanDrupal\Rules\Drupal\LoadIncludes
 	- mglaman\PHPStanDrupal\Rules\Drupal\EntityQuery\EntityQueryHasAccessCheckRule
+	- mglaman\PHPStanDrupal\Rules\Deprecations\SymfonyCmfRouteObjectInterfaceConstantsRule

src/Internal/DeprecatedScopeCheck.php

@@ -0,0 +1,22 @@
+<?php declare(strict_types=1);
+
+namespace mglaman\PHPStanDrupal\Internal;
+
+use PHPStan\Analyser\Scope;
+
+final class DeprecatedScopeCheck
+{
+    public static function inDeprecatedScope(Scope $scope): bool
+    {
+        $class = $scope->getClassReflection();
+        if ($class !== null && $class->isDeprecated()) {
+            return true;
+        }
+        $trait = $scope->getTraitReflection();
+        if ($trait !== null && $trait->isDeprecated()) {
+            return true;
+        }
+        $function = $scope->getFunction();
+        return $function !== null && $function->isDeprecated()->yes();
+    }
+}

src/Rules/Deprecations/AccessDeprecatedConstant.php

@@ -2,9 +2,9 @@
 
 namespace mglaman\PHPStanDrupal\Rules\Deprecations;
 
+use mglaman\PHPStanDrupal\Internal\DeprecatedScopeCheck;
 use PhpParser\Node;
 use PHPStan\Analyser\Scope;
-use PHPStan\Reflection\FunctionReflection;
 use PHPStan\Reflection\ReflectionProvider;
 
 class AccessDeprecatedConstant implements \PHPStan\Rules\Rule
@@ -24,16 +24,7 @@ public function getNodeType(): string
     public function processNode(Node $node, Scope $scope): array
     {
         assert($node instanceof Node\Expr\ConstFetch);
-        $class = $scope->getClassReflection();
-        if ($class !== null && $class->isDeprecated()) {
-            return [];
-        }
-        $trait = $scope->getTraitReflection();
-        if ($trait !== null && $trait->isDeprecated()) {
-            return [];
-        }
-        $function = $scope->getFunction();
-        if ($function instanceof FunctionReflection && $function->isDeprecated()->yes()) {
+        if (DeprecatedScopeCheck::inDeprecatedScope($scope)) {
             return [];
         }
 

src/Rules/Deprecations/SymfonyCmfRouteObjectInterfaceConstantsRule.php

@@ -0,0 +1,63 @@
+<?php declare(strict_types=1);
+
+namespace mglaman\PHPStanDrupal\Rules\Deprecations;
+
+use Drupal\Core\Routing\RouteObjectInterface;
+use mglaman\PHPStanDrupal\Internal\DeprecatedScopeCheck;
+use PhpParser\Node;
+use PHPStan\Analyser\Scope;
+use PHPStan\Rules\Rule;
+use PHPStan\Rules\RuleErrorBuilder;
+use PHPStan\Type\ObjectType;
+
+final class SymfonyCmfRouteObjectInterfaceConstantsRule implements Rule
+{
+
+    public function getNodeType(): string
+    {
+        return Node\Expr\ClassConstFetch::class;
+    }
+
+    public function processNode(Node $node, Scope $scope): array
+    {
+        assert($node instanceof Node\Expr\ClassConstFetch);
+        if (!$node->name instanceof Node\Identifier) {
+            return [];
+        }
+        if (!$node->class instanceof Node\Name) {
+            return [];
+        }
+        $constantName = $node->name->name;
+        $className = $node->class;
+        $classType = $scope->resolveTypeByName($className);
+        if (!$classType->hasConstant($constantName)->yes()) {
+            return [];
+        }
+        if (DeprecatedScopeCheck::inDeprecatedScope($scope)) {
+            return [];
+        }
+        [$major, $minor] = explode('.', \Drupal::VERSION, 3);
+        if ($major !== '9' && (int) $minor > 1) {
+            return [];
+        }
+        $cmfRouteObjectInterfaceType = new ObjectType(\Symfony\Cmf\Component\Routing\RouteObjectInterface::class);
+        if (!$classType->isSuperTypeOf($cmfRouteObjectInterfaceType)->yes()) {
+            return [];
+        }
+
+        $coreRouteObjectInterfaceType = new ObjectType(RouteObjectInterface::class);
+        if (!$coreRouteObjectInterfaceType->hasConstant($constantName)->yes()) {
+            return [
+                RuleErrorBuilder::message(
+                    sprintf('The core dependency symfony-cmf/routing is deprecated and %s::%s is not supported.', $className, $constantName)
+                )->tip('Change record: https://www.drupal.org/node/3151009')->build(),
+            ];
+        }
+
+        return [
+            RuleErrorBuilder::message(
+                sprintf('%s::%s is deprecated and removed in Drupal 10. Use \Drupal\Core\Routing\RouteObjectInterface::%2$s instead.', $className, $constantName)
+            )->tip('Change record: https://www.drupal.org/node/3151009')->build(),
+        ];
+    }
+}

tests/src/Rules/SymfonyCmfRouteObjectInterfaceConstantsRuleTest.php

@@ -0,0 +1,46 @@
+<?php
+declare(strict_types=1);
+
+namespace mglaman\PHPStanDrupal\Tests\Rules;
+
+use mglaman\PHPStanDrupal\Rules\Deprecations\SymfonyCmfRouteObjectInterfaceConstantsRule;
+use mglaman\PHPStanDrupal\Tests\DrupalRuleTestCase;
+
+final class SymfonyCmfRouteObjectInterfaceConstantsRuleTest extends DrupalRuleTestCase
+{
+
+    protected function getRule(): \PHPStan\Rules\Rule
+    {
+        return new SymfonyCmfRouteObjectInterfaceConstantsRule();
+    }
+
+    public function testRule(): void
+    {
+        $this->analyse(
+            [__DIR__.'/data/symfony-cmf-routing.php'],
+            [
+                [
+                    'Symfony\Cmf\Component\Routing\RouteObjectInterface::ROUTE_NAME is deprecated and removed in Drupal 10. Use \Drupal\Core\Routing\RouteObjectInterface::ROUTE_NAME instead.',
+                    6,
+                    'Change record: https://www.drupal.org/node/3151009'
+                ],
+                [
+                    'Symfony\Cmf\Component\Routing\RouteObjectInterface::ROUTE_OBJECT is deprecated and removed in Drupal 10. Use \Drupal\Core\Routing\RouteObjectInterface::ROUTE_OBJECT instead.',
+                    7,
+                    'Change record: https://www.drupal.org/node/3151009'
+                ],
+                [
+                    'Symfony\Cmf\Component\Routing\RouteObjectInterface::CONTROLLER_NAME is deprecated and removed in Drupal 10. Use \Drupal\Core\Routing\RouteObjectInterface::CONTROLLER_NAME instead.',
+                    8,
+                    'Change record: https://www.drupal.org/node/3151009'
+                ],
+                [
+                    'The core dependency symfony-cmf/routing is deprecated and Symfony\Cmf\Component\Routing\RouteObjectInterface::TEMPLATE_NAME is not supported.',
+                    9,
+                    'Change record: https://www.drupal.org/node/3151009'
+                ],
+            ]
+        );
+    }
+
+}

tests/src/Rules/data/symfony-cmf-routing.php

@@ -0,0 +1,33 @@
+<?php
+
+namespace SymfonyCmfRoutingUsage;
+
+class Foo {
+    public const NAME = \Symfony\Cmf\Component\Routing\RouteObjectInterface::ROUTE_NAME;
+    public const OBJECT = \Symfony\Cmf\Component\Routing\RouteObjectInterface::ROUTE_OBJECT;
+    public const CONTROLLER = \Symfony\Cmf\Component\Routing\RouteObjectInterface::CONTROLLER_NAME;
+    public const UNMAPPED = \Symfony\Cmf\Component\Routing\RouteObjectInterface::TEMPLATE_NAME;
+    public function a(\Symfony\Cmf\Component\Routing\RouteObjectInterface $object) {
+
+    }
+    public function b(\Symfony\Cmf\Component\Routing\RouteProviderInterface $provider) {
+
+    }
+    public function c(\Symfony\Cmf\Component\Routing\LazyRouteCollection $collection) {
+
+    }
+}
+class Bar {
+    public const NAME = \Drupal\Core\Routing\RouteObjectInterface::ROUTE_NAME;
+    public const OBJECT = \Drupal\Core\Routing\RouteObjectInterface::ROUTE_OBJECT;
+    public const CONTROLLER = \Drupal\Core\Routing\RouteObjectInterface::CONTROLLER_NAME;
+    public function a(\Drupal\Core\Routing\RouteObjectInterface $object) {
+
+    }
+    public function b(\Drupal\Core\Routing\RouteProviderInterface $provider) {
+
+    }
+    public function c(\Drupal\Core\Routing\LazyRouteCollection $collection) {
+
+    }
+}