Merge branch 'hotfix/1.3.1'

CHANGELOG.md

@@ -6,7 +6,13 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ## [Unreleased]
 
-## v1.3.0
+## v1.3.1 (2022-11-17)
+
+### Fixed
+
+- Fix `twig_toolkit_url` function throwing an error when used with a `null` value ([#16](https://github.com/studiometa/twig-toolkit/pull/16))
+
+## v1.3.0 (2022-11-17)
 
 ### Added
 

composer.json

@@ -1,6 +1,6 @@
 {
   "name": "studiometa/twig-toolkit",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "A set of useful extension and components for Twig.",
   "license": "MIT",
   "require": {

src/Extension.php

@@ -9,15 +9,14 @@
 
 use Studiometa\TwigToolkit\Helpers\Html;
 use Studiometa\TwigToolkit\Helpers\Template;
+use Studiometa\TwigToolkit\Helpers\Url;
 use Studiometa\TwigToolkit\Node\ElementNode;
 use Studiometa\TwigToolkit\TokenParser\ElementTokenParser;
-
 use Twig\Extension\AbstractExtension;
 use Twig\Loader\FilesystemLoader;
 use Twig\TokenParser\TokenParserInterface;
-use Twig\TwigFunction;
 use Twig\TwigFilter;
-use Spatie\Url\Url;
+use Twig\TwigFunction;
 
 /**
  * Twig extension class.

src/Helpers/Url.php

@@ -0,0 +1,28 @@
+<?php
+/**
+ * @link https://github.com/studiometa/twig-toolkit
+ * @copyright Studio Meta
+ * @license https://github.com/studiometa/twig-toolkit/blob/master/LICENSE
+ */
+
+namespace Studiometa\TwigToolkit\Helpers;
+
+use Twig\Environment;
+use Spatie\Url\Url as UrlCore;
+
+/**
+ * Url manipulation helper.
+ *
+ * @author Studio Meta <[email protected]>
+ * @since 1.3.1
+ */
+class Url extends UrlCore
+{
+    /**
+     * @return self
+     */
+    public static function fromString(?string $url)
+    {
+        return parent::fromString($url ?? '');
+    }
+}

tests/Helpers/UrlTest.php

@@ -0,0 +1,13 @@
+<?php
+
+use function Spatie\Snapshots\assertMatchesSnapshot;
+use Studiometa\TwigToolkit\Helpers\Url;
+
+test('The `Url` class should work with null given as $url', function () {
+    expect(function () {
+        Url::fromString(null);
+    })->not->toThrow('type string, null given');
+
+    expect((string)Url::fromString(null))->toBe('');
+    expect((string)Url::fromString(null)->withHost('fqdn.tld'))->toBe('//fqdn.tld');
+});