[Icons] fix set default viewBox when not defined in Iconify icon set

Author: stlgaits

src/Icons/src/Iconify.php

@@ -82,7 +82,8 @@ public function fetchIcon(string $prefix, string $name): Icon
         $height = $data['icons'][$name]['height'] ?? $data['height'] ?? $this->sets()[$prefix]['height'] ?? null;
         $width = $data['icons'][$name]['width'] ?? $data['width'] ?? $this->sets()[$prefix]['width'] ?? null;
         if (null === $width && null === $height) {
-            throw new \RuntimeException(\sprintf('The icon "%s:%s" does not have a width or height.', $prefix, $nameArg));
+            $height = 16;
+            $width = 16;
         }
 
         return new Icon($data['icons'][$name]['body'], [
@@ -133,6 +134,11 @@ public function fetchIcons(string $prefix, array $names): array
             $height = $iconData['height'] ?? $data['height'] ??= $this->sets()[$prefix]['height'] ?? null;
             $width = $iconData['width'] ?? $data['width'] ??= $this->sets()[$prefix]['width'] ?? null;
 
+            if (null === $width && null === $height) {
+                $height = 16;
+                $width = 16;
+            }
+
             $icons[$iconName] = new Icon($iconData['body'], [
                 'xmlns' => self::ATTR_XMLNS_URL,
                 'viewBox' => \sprintf('0 0 %d %d', $width ?? $height, $height ?? $width),

src/Icons/tests/Unit/IconifyTest.php

@@ -119,7 +119,7 @@ public function testFetchIconUsesIconsetViewBoxHeight()
         $this->assertEquals('0 0 17 17', $icon->getAttributes()['viewBox']);
     }
 
-    public function testFetchIconThrowsWhenViewBoxCannotBeComputed()
+    public function testFetchIconSetsDefaultViewBoxTo16()
     {
         $iconify = new Iconify(
             cache: new NullAdapter(),
@@ -138,10 +138,11 @@ public function testFetchIconThrowsWhenViewBoxCannotBeComputed()
             ]),
         );
 
-        $this->expectException(\RuntimeException::class);
-        $this->expectExceptionMessage('The icon "bi:heart" does not have a width or height.');
+        $icon = $iconify->fetchIcon('bi', 'heart');
 
-        $iconify->fetchIcon('bi', 'heart');
+        $this->assertIsArray($icon->getAttributes());
+        $this->assertArrayHasKey('viewBox', $icon->getAttributes());
+        $this->assertEquals('0 0 16 16', $icon->getAttributes()['viewBox']);
     }
 
     public function testFetchIconThrowsWhenStatusCodeNot200()