Fixed trimming null raising TypeError; now returns empty string.

ScriptFUSION · Dec 14, 2024 · 7292fdb · 7292fdb
1 parent 45aa554
commit 7292fdb
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/Strategy/Trim.php b/src/Strategy/Trim.php
@@ -7,6 +7,6 @@ class Trim extends Delegate
 {
     public function __invoke($data, $context = null)
     {
-        return trim(parent::__invoke($data, $context));
+        return trim((string)parent::__invoke($data, $context));
     }
 }
diff --git a/test/Integration/Mapper/Strategy/TrimTest.php b/test/Integration/Mapper/Strategy/TrimTest.php
@@ -18,4 +18,14 @@ public function test(): void
 
         self::assertSame('foo', $trim([]));
     }
+
+    /**
+     * Tests that trimming null returns the empty string.
+     */
+    public function testNull(): void
+    {
+        $trim = (new Trim(null))->setMapper(new Mapper());
+
+        self::assertSame('', $trim([]));
+    }
 }