Add analysis for route generation by controllers and UrlGeneratorInterface

Author: Ismail Turan

composer.json

@@ -33,6 +33,7 @@
     "symfony/http-foundation": "^5.1",
     "symfony/messenger": "^4.2 || ^5.0",
     "symfony/polyfill-php80": "^1.24",
+    "symfony/routing": "^4.4 || ^5.0",
     "symfony/serializer": "^4.0 || ^5.0"
   },
   "config": {

extension.neon

@@ -11,6 +11,8 @@ parameters:
 		constantHassers: true
 		console_application_loader: null
 		consoleApplicationLoader: null
+		url_generating_rules_file: null
+		urlGeneratingRulesFile: null
 	stubFiles:
 		- stubs/Psr/Cache/CacheItemInterface.stub
 		- stubs/Symfony/Bundle/FrameworkBundle/KernelBrowser.stub
@@ -65,6 +67,8 @@ parametersSchema:
 		constantHassers: bool()
 		console_application_loader: schema(string(), nullable())
 		consoleApplicationLoader: schema(string(), nullable())
+		url_generating_rules_file: schema(string(), nullable())
+		urlGeneratingRulesFile: schema(string(), nullable())
 	])
 
 services:
@@ -89,6 +93,13 @@ services:
 	-
 		factory: @symfony.parameterMapFactory::create()
 
+	# url generating routes map
+	symfony.urlGeneratingRoutesMapFactory:
+		class: PHPStan\Symfony\UrlGeneratingRoutesMapFactory
+		factory: PHPStan\Symfony\PhpUrlGeneratingRoutesMapFactory
+	-
+		factory: @symfony.urlGeneratingRoutesMapFactory::create()
+
 	# ControllerTrait::get()/has() return type
 	-
 		factory: PHPStan\Type\Symfony\ServiceDynamicReturnTypeExtension(Symfony\Component\DependencyInjection\ContainerInterface)

rules.neon

@@ -5,4 +5,5 @@ rules:
 	- PHPStan\Rules\Symfony\InvalidArgumentDefaultValueRule
 	- PHPStan\Rules\Symfony\UndefinedOptionRule
 	- PHPStan\Rules\Symfony\InvalidOptionDefaultValueRule
+	- PHPStan\Rules\Symfony\UrlGeneratorInterfaceUnknownRouteRule
 

src/Rules/Symfony/UrlGeneratorInterfaceUnknownRouteRule.php

@@ -0,0 +1,75 @@
+<?php declare(strict_types = 1);
+
+namespace PHPStan\Rules\Symfony;
+
+use PhpParser\Node;
+use PhpParser\Node\Expr\MethodCall;
+use PHPStan\Analyser\Scope;
+use PHPStan\Rules\Rule;
+use PHPStan\ShouldNotHappenException;
+use PHPStan\Symfony\UrlGeneratingRoutesMap;
+use PHPStan\Type\ObjectType;
+
+/**
+ * @implements Rule<MethodCall>
+ */
+final class UrlGeneratorInterfaceUnknownRouteRule implements Rule
+{
+
+	/** @var UrlGeneratingRoutesMap */
+	private $urlGeneratingRoutesMap;
+
+	public function __construct(UrlGeneratingRoutesMap $urlGeneratingRoutesMap)
+	{
+		$this->urlGeneratingRoutesMap = $urlGeneratingRoutesMap;
+	}
+
+	public function getNodeType(): string
+	{
+		return MethodCall::class;
+	}
+
+	/**
+	 * @param \PhpParser\Node $node
+	 * @param \PHPStan\Analyser\Scope $scope
+	 * @return (string|\PHPStan\Rules\RuleError)[] errors
+	 */
+	public function processNode(Node $node, Scope $scope): array
+	{
+		if (!$node instanceof MethodCall) {
+			throw new ShouldNotHappenException();
+		}
+
+		if (!$node->name instanceof Node\Identifier) {
+			return [];
+		}
+
+		if (in_array($node->name->name, ['generate', 'generateUrl'], true) === false || !isset($node->getArgs()[0])) {
+			return [];
+		}
+
+		$argType = $scope->getType($node->var);
+		$isControllerType = (new ObjectType('Symfony\Bundle\FrameworkBundle\Controller\Controller'))->isSuperTypeOf($argType);
+		$isAbstractControllerType = (new ObjectType('Symfony\Bundle\FrameworkBundle\Controller\AbstractController'))->isSuperTypeOf($argType);
+		$isUrlGeneratorInterface = (new ObjectType('Symfony\Component\Routing\Generator\UrlGeneratorInterface'))->isSuperTypeOf($argType);
+		if (
+			$isControllerType->no()
+			&& $isAbstractControllerType->no()
+			&& $isUrlGeneratorInterface->no()
+		) {
+			return [];
+		}
+
+		$routeName = $this->urlGeneratingRoutesMap::getRouteNameFromNode($node->getArgs()[0]->value, $scope);
+		if ($routeName === null) {
+			return [];
+		}
+
+		if ($this->urlGeneratingRoutesMap->hasRouteName($routeName) === false) {
+			return [sprintf('Route with name "%s" does not exist.', $routeName)];
+		}
+
+		return [];
+	}
+
+}

src/Symfony/Configuration.php

@@ -31,4 +31,9 @@ public function getConsoleApplicationLoader(): ?string
 		return $this->parameters['consoleApplicationLoader'] ?? $this->parameters['console_application_loader'] ?? null;
 	}
 
+	public function getUrlGeneratingRoutesFile(): ?string
+	{
+		return $this->parameters['urlGeneratingRulesFile'] ?? $this->parameters['url_generating_rules_file'] ?? null;
+	}
+
 }

src/Symfony/DefaultUrlGeneratingRoutesMap.php

@@ -0,0 +1,41 @@
+<?php declare(strict_types = 1);
+
+namespace PHPStan\Symfony;
+
+use PhpParser\Node\Expr;
+use PHPStan\Analyser\Scope;
+use PHPStan\Type\TypeUtils;
+use function count;
+
+final class DefaultUrlGeneratingRoutesMap implements UrlGeneratingRoutesMap
+{
+
+	/** @var \PHPStan\Symfony\UrlGeneratingRoutesDefinition[] */
+	private $routes;
+
+	/**
+	 * @param \PHPStan\Symfony\UrlGeneratingRoutesDefinition[] $routes
+	 */
+	public function __construct(array $routes)
+	{
+		$this->routes = $routes;
+	}
+
+	public function hasRouteName(string $name): bool
+	{
+		foreach ($this->routes as $route) {
+			if ($route->getName() === $name) {
+				return true;
+			}
+		}
+
+		return false;
+	}
+
+	public static function getRouteNameFromNode(Expr $node, Scope $scope): ?string
+	{
+		$strings = TypeUtils::getConstantStrings($scope->getType($node));
+		return count($strings) === 1 ? $strings[0]->getValue() : null;
+	}
+
+}

src/Symfony/FakeUrlGeneratingRoutesMap.php

@@ -0,0 +1,21 @@
+<?php declare(strict_types = 1);
+
+namespace PHPStan\Symfony;
+
+use PhpParser\Node\Expr;
+use PHPStan\Analyser\Scope;
+
+final class FakeUrlGeneratingRoutesMap implements UrlGeneratingRoutesMap
+{
+
+	public function hasRouteName(string $name): bool
+	{
+		return false;
+	}
+
+	public static function getRouteNameFromNode(Expr $node, Scope $scope): ?string
+	{
+		return null;
+	}
+
+}

src/Symfony/PhpUrlGeneratingRoutesMapFactory.php

@@ -0,0 +1,51 @@
+<?php declare(strict_types = 1);
+
+namespace PHPStan\Symfony;
+
+use function sprintf;
+
+final class PhpUrlGeneratingRoutesMapFactory implements UrlGeneratingRoutesMapFactory
+{
+
+	/** @var string|null */
+	private $urlGeneratingRoutesFile;
+
+	public function __construct(Configuration $configuration)
+	{
+		$this->urlGeneratingRoutesFile = $configuration->getUrlGeneratingRoutesFile();
+	}
+
+	public function create(): UrlGeneratingRoutesMap
+	{
+		if ($this->urlGeneratingRoutesFile === null) {
+			return new FakeUrlGeneratingRoutesMap();
+		}
+
+		if (file_exists($this->urlGeneratingRoutesFile) === false) {
+			throw new UrlGeneratingRoutesFileNotExistsException(sprintf('File %s containing route generator information does not exist.', $this->urlGeneratingRoutesFile));
+		}
+
+		$urlGeneratingRoutes = require $this->urlGeneratingRoutesFile;
+
+		if (!is_array($urlGeneratingRoutes)) {
+			throw new UrlGeneratingRoutesFileNotExistsException(sprintf('File %s containing route generator information cannot be parsed.', $this->urlGeneratingRoutesFile));
+		}
+
+		/** @var \PHPStan\Symfony\UrlGeneratingRoutesDefinition[] $routes */
+		$routes = [];
+		foreach ($urlGeneratingRoutes as $routeName => $routeConfiguration) {
+			if (!is_string($routeName)) {
+				continue;
+			}
+
+			if (!is_array($routeConfiguration) || !isset($routeConfiguration[1]['_controller'])) {
+				continue;
+			}
+
+			$routes[] = new UrlGeneratingRoute($routeName, $routeConfiguration[1]['_controller']);
+		}
+
+		return new DefaultUrlGeneratingRoutesMap($routes);
+	}
+
+}

src/Symfony/UrlGeneratingRoute.php

@@ -0,0 +1,33 @@
+<?php declare(strict_types = 1);
+
+namespace PHPStan\Symfony;
+
+class UrlGeneratingRoute implements UrlGeneratingRoutesDefinition
+{
+
+	/** @var string */
+	private $name;
+
+	/** @var string */
+	private $controller;
+
+	public function __construct(
+		string $name,
+		string $controller
+	)
+	{
+		$this->name = $name;
+		$this->controller = $controller;
+	}
+
+	public function getName(): string
+	{
+		return $this->name;
+	}
+
+	public function getController(): ?string
+	{
+		return $this->controller;
+	}
+
+}

src/Symfony/UrlGeneratingRoutesDefinition.php

@@ -0,0 +1,12 @@
+<?php declare(strict_types = 1);
+
+namespace PHPStan\Symfony;
+
+interface UrlGeneratingRoutesDefinition
+{
+
+	public function getName(): string;
+
+	public function getController(): ?string;
+
+}