fabbot

weaverryan · weaverryan · commit 26d0d028d417 · 2023-04-28T16:41:16.000-04:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php
@@ -836,20 +836,21 @@ private function addAssetMapperSection(ArrayNodeDefinition $rootNode, callable $
                                     $result = [];
                                     foreach ($v as $key => $item) {
                                         // "dir" => "namespace"
-                                        if (is_string($key)) {
+                                        if (\is_string($key)) {
                                             $result[$key] = $item;
 
                                             continue;
                                         }
 
-                                        if (is_array($item)) {
+                                        if (\is_array($item)) {
                                             // $item = ["namespace" => "the/namespace", "value" => "the/dir"]
-                                            $result[$item['value']] = isset($item['namespace']) ? $item['namespace'] : '';
+                                            $result[$item['value']] = $item['namespace'] ?? '';
                                         } else {
                                             // $item = "the/dir"
                                             $result[$item] = '';
                                         }
                                     }
+
                                     return $result;
                                 })
                             ->end()
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/XmlFrameworkExtensionTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/XmlFrameworkExtensionTest.php
@@ -14,7 +14,6 @@
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
-use Symfony\Component\RateLimiter\Policy\SlidingWindowLimiter;
 
 class XmlFrameworkExtensionTest extends FrameworkExtensionTestCase
 {
diff --git a/src/Symfony/Component/AssetMapper/AssetDependency.php b/src/Symfony/Component/AssetMapper/AssetDependency.php
@@ -19,7 +19,7 @@
 final class AssetDependency
 {
     /**
-     * @param bool $isLazy Whether this dependency is immediately needed.
+     * @param bool $isLazy whether this dependency is immediately needed
      */
     public function __construct(
         public readonly MappedAsset $asset,
diff --git a/src/Symfony/Component/AssetMapper/AssetMapper.php b/src/Symfony/Component/AssetMapper/AssetMapper.php
@@ -15,6 +15,7 @@
  * Finds and returns assets in the pipeline.
  *
  * @experimental
+ *
  * @final
  */
 class AssetMapper implements AssetMapperInterface
diff --git a/src/Symfony/Component/AssetMapper/AssetMapperCompiler.php b/src/Symfony/Component/AssetMapper/AssetMapperCompiler.php
@@ -17,6 +17,7 @@
  * Runs a chain of compiles intended to adjust the source of assets.
  *
  * @experimental
+ *
  * @final
  */
 class AssetMapperCompiler
diff --git a/src/Symfony/Component/AssetMapper/AssetMapperInterface.php b/src/Symfony/Component/AssetMapper/AssetMapperInterface.php
@@ -22,8 +22,6 @@ interface AssetMapperInterface
 {
     /**
      * The path that should be prefixed on all asset paths to point to the output location.
-     *
-     * @return string
      */
     public function getPublicPrefix(): string;
 
diff --git a/src/Symfony/Component/AssetMapper/AssetMapperRepository.php b/src/Symfony/Component/AssetMapper/AssetMapperRepository.php
@@ -20,6 +20,7 @@
  * @experimental
  *
  * @author Ryan Weaver <ryan@symfonycasts.com>
+ *
  * @final
  */
 class AssetMapperRepository

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@`
`14`	`14`	`use Symfony\Component\Config\FileLocator;`
`15`	`15`	`use Symfony\Component\DependencyInjection\ContainerBuilder;`
`16`	`16`	`use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;`
`17`		`-use Symfony\Component\RateLimiter\Policy\SlidingWindowLimiter;`
`18`	`17`
`19`	`18`	`class XmlFrameworkExtensionTest extends FrameworkExtensionTestCase`
`20`	`19`	`{`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@`
`19`	`19`	`final class AssetDependency`
`20`	`20`	`{`
`21`	`21`	`/**`
`22`		`- * @param bool $isLazy Whether this dependency is immediately needed.`
	`22`	`+ * @param bool $isLazy whether this dependency is immediately needed`
`23`	`23`	`*/`
`24`	`24`	`public function __construct(`
`25`	`25`	`public readonly MappedAsset $asset,`
Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,7 @@`
`15`	`15`	`* Finds and returns assets in the pipeline.`
`16`	`16`	`*`
`17`	`17`	`* @experimental`
	`18`	`+ *`
`18`	`19`	`* @final`
`19`	`20`	`*/`
`20`	`21`	`class AssetMapper implements AssetMapperInterface`
Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@`
`17`	`17`	`* Runs a chain of compiles intended to adjust the source of assets.`
`18`	`18`	`*`
`19`	`19`	`* @experimental`
	`20`	`+ *`
`20`	`21`	`* @final`
`21`	`22`	`*/`
`22`	`23`	`class AssetMapperCompiler`
Original file line number	Diff line number	Diff line change
`@@ -22,8 +22,6 @@ interface AssetMapperInterface`
`22`	`22`	`{`
`23`	`23`	`/**`
`24`	`24`	`* The path that should be prefixed on all asset paths to point to the output location.`
`25`		`- *`
`26`		`- * @return string`
`27`	`25`	`*/`
`28`	`26`	`public function getPublicPrefix(): string;`
`29`	`27`
Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,7 @@`
`20`	`20`	`* @experimental`
`21`	`21`	`*`
`22`	`22`	`* @author Ryan Weaver <ryan@symfonycasts.com>`
	`23`	`+ *`
`23`	`24`	`* @final`
`24`	`25`	`*/`
`25`	`26`	`class AssetMapperRepository`