Apply suggestions from code review

derrabus · nicolas-grekas · web-flow · commit 2da4ceedcdf3 · 2023-10-13T15:24:30.000+02:00
Co-authored-by: Nicolas Grekas &lt;nicolas.grekas@gmail.com&gt;
diff --git a/src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/RegisterMappingsPass.php b/src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/RegisterMappingsPass.php
@@ -112,7 +112,7 @@ public function __construct(
         $this->driverPattern = $driverPattern;
         $this->enabledParameter = $enabledParameter;
 
-        if (\count($aliasMap) && (!$configurationPattern || !$registerAliasMethodName)) {
+        if ($aliasMap && (!$configurationPattern || !$registerAliasMethodName)) {
             throw new \InvalidArgumentException('configurationPattern and registerAliasMethodName are required to register namespace alias.');
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,7 @@ public function __construct(`
`112`	`112`	`$this->driverPattern = $driverPattern;`
`113`	`113`	`$this->enabledParameter = $enabledParameter;`
`114`	`114`
`115`		`- if (\count($aliasMap) && (!$configurationPattern \|\| !$registerAliasMethodName)) {`
	`115`	`+ if ($aliasMap && (!$configurationPattern \|\| !$registerAliasMethodName)) {`
`116`	`116`	`throw new \InvalidArgumentException('configurationPattern and registerAliasMethodName are required to register namespace alias.');`
`117`	`117`	`}`
`118`	`118`	`}`