refactor: better handling of empty asset

src/Assets/Asset.php

@@ -59,7 +59,7 @@ class Asset implements \ArrayAccess
      *
      * @throws RuntimeException
      */
-    public function __construct(Builder $builder, $paths, array $options = null)
+    public function __construct(Builder $builder, string|array $paths, array|null $options = null)
     {
         $this->builder = $builder;
         $this->config = $builder->getConfig();

src/Renderer/Extension/Core.php

@@ -280,8 +280,12 @@ public function url(array $context, $value = null, array $options = null): strin
      *
      * @return Asset
      */
-    public function asset($path, array $options = null): Asset
+    public function asset($path, array|null $options = null): Asset
     {
+        if (!\is_string($path) && !\is_array($path)) {
+            throw new RuntimeException(sprintf('Argument of "%s()" must a string or an array.', \Cecil\Util::formatMethodName(__METHOD__)));
+        }
+
         return new Asset($this->builder, $path, $options);
     }
 

src/Util.php

@@ -37,6 +37,20 @@ public static function formatClassName($class, array $options = []): string
         return $className;
     }
 
+    /**
+     * Formats a method name.
+     *
+     * ie: "Cecil\Renderer\Extension\Core::asset()" become "asset()"
+     *
+     * @param string $method
+     */
+    public static function formatMethodName(string $method): string
+    {
+        $methodName = explode('::', $method)[1];
+
+        return $methodName;
+    }
+
     /**
      * Converts an array of strings into a path.
      */