Enrich Router's MissingMandatoryParametersException

adrienlucas · adrienlucas · commit 22092a1ef890 · 2022-01-19T12:12:54.000+01:00
diff --git a/src/Symfony/Component/Routing/CHANGELOG.md b/src/Symfony/Component/Routing/CHANGELOG.md
@@ -4,6 +4,7 @@ CHANGELOG
 6.1
 ---
 
+ * Added `getMissingParameters` and `getRouteName` methods on `MissingMandatoryParametersException`
  * Allow using UTF-8 parameter names
 
 5.3
diff --git a/src/Symfony/Component/Routing/Exception/MissingMandatoryParametersException.php b/src/Symfony/Component/Routing/Exception/MissingMandatoryParametersException.php
@@ -19,4 +19,36 @@
  */
 class MissingMandatoryParametersException extends \InvalidArgumentException implements ExceptionInterface
 {
+    private array $missingParameters = [];
+    private string $routeName = '';
+
+    public function __construct(/*array*/ $missingParameters, /*string*/ $routeName, /*int*/ $code = 0, \Throwable $previous = null)
+    {
+        if(\is_array($this->missingParameters)) {
+            $this->missingParameters = $missingParameters;
+            $this->routeName = $routeName;
+            $message = sprintf(
+                'Some mandatory parameters are missing ("%s") to generate a URL for route "%s".',
+                implode('", "', $missingParameters),
+                $routeName
+            );
+        } else {
+            trigger_deprecation('symfony/routing', '6.1', 'Construction of "%s" with an exception message is deprecated, provide an array of missing parameters and the route name instead.', __CLASS__);
+            $message = (string) $missingParameters;
+            $code = (int) $routeName;
+            $previous = $code instanceof \Throwable ? $code : null;
+        }
+
+        parent::__construct($message, $code, $previous);
+    }
+
+    public function getMissingParameters(): array
+    {
+        return $this->missingParameters;
+    }
+
+    public function getRouteName(): string
+    {
+        return $this->routeName;
+    }
 }
diff --git a/src/Symfony/Component/Routing/Generator/UrlGenerator.php b/src/Symfony/Component/Routing/Generator/UrlGenerator.php
@@ -171,7 +171,7 @@ protected function doGenerate(array $variables, array $defaults, array $requirem
 
         // all params must be given
         if ($diff = array_diff_key($variables, $mergedParams)) {
-            throw new MissingMandatoryParametersException(sprintf('Some mandatory parameters are missing ("%s") to generate a URL for route "%s".', implode('", "', array_keys($diff)), $name));
+            throw new MissingMandatoryParametersException(array_keys($diff), $name);
         }
 
         $url = '';
diff --git a/src/Symfony/Component/Routing/Tests/Generator/UrlGeneratorTest.php b/src/Symfony/Component/Routing/Tests/Generator/UrlGeneratorTest.php
@@ -324,6 +324,7 @@ public function testGenerateWithInvalidLocale()
     public function testGenerateForRouteWithoutMandatoryParameter()
     {
         $this->expectException(MissingMandatoryParametersException::class);
+        $this->expectExceptionMessage('Some mandatory parameters are missing ("foo") to generate a URL for route "test".');
         $routes = $this->getRoutes('test', new Route('/testing/{foo}'));
         $this->getGenerator($routes)->generate('test', [], UrlGeneratorInterface::ABSOLUTE_URL);
     }
@@ -554,6 +555,7 @@ public function testImportantVariable()
     public function testImportantVariableWithNoDefault()
     {
         $this->expectException(MissingMandatoryParametersException::class);
+        $this->expectExceptionMessage('Some mandatory parameters are missing ("_format") to generate a URL for route "test".');
         $routes = $this->getRoutes('test', new Route('/{page}.{!_format}'));
         $generator = $this->getGenerator($routes);
 

Original file line number	Diff line number	Diff line change
`@@ -171,7 +171,7 @@ protected function doGenerate(array $variables, array $defaults, array $requirem`
`171`	`171`
`172`	`172`	`// all params must be given`
`173`	`173`	`if ($diff = array_diff_key($variables, $mergedParams)) {`
`174`		`- throw new MissingMandatoryParametersException(sprintf('Some mandatory parameters are missing ("%s") to generate a URL for route "%s".', implode('", "', array_keys($diff)), $name));`
	`174`	`+ throw new MissingMandatoryParametersException(array_keys($diff), $name);`
`175`	`175`	`}`
`176`	`176`
`177`	`177`	`$url = '';`
Original file line number	Diff line number	Diff line change
`@@ -324,6 +324,7 @@ public function testGenerateWithInvalidLocale()`
`324`	`324`	`public function testGenerateForRouteWithoutMandatoryParameter()`
`325`	`325`	`{`
`326`	`326`	`$this->expectException(MissingMandatoryParametersException::class);`
	`327`	`+ $this->expectExceptionMessage('Some mandatory parameters are missing ("foo") to generate a URL for route "test".');`
`327`	`328`	`$routes = $this->getRoutes('test', new Route('/testing/{foo}'));`
`328`	`329`	`$this->getGenerator($routes)->generate('test', [], UrlGeneratorInterface::ABSOLUTE_URL);`
`329`	`330`	`}`
`@@ -554,6 +555,7 @@ public function testImportantVariable()`
`554`	`555`	`public function testImportantVariableWithNoDefault()`
`555`	`556`	`{`
`556`	`557`	`$this->expectException(MissingMandatoryParametersException::class);`
	`558`	`+ $this->expectExceptionMessage('Some mandatory parameters are missing ("_format") to generate a URL for route "test".');`
`557`	`559`	`$routes = $this->getRoutes('test', new Route('/{page}.{!_format}'));`
`558`	`560`	`$generator = $this->getGenerator($routes);`
`559`	`561`