Merge pull request #68 from Crell/json-stream-quote

Escape strings in JSON stream formatter.
Crell · Aug 2, 2024 · 6c1a862 · 6c1a862
2 parents d9f47ea + 1cc3151
commit 6c1a862
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/src/Formatter/JsonStreamFormatter.php b/src/Formatter/JsonStreamFormatter.php
@@ -22,6 +22,15 @@ public function format(): string
         return 'json-stream';
     }
 
+    /**
+     * @param FormatterStream $runningValue
+     */
+    public function serializeString(mixed $runningValue, Field $field, ?string $next): mixed
+    {
+        $runningValue->printf('"%s"', is_string($next) ? str_replace('"', '\"', $next) : null);
+        return $runningValue;
+    }
+
     /**
      * @param FormatterStream $runningValue
      */

diff --git a/tests/JsonStreamFormatterTest.php b/tests/JsonStreamFormatterTest.php
@@ -154,6 +154,12 @@ public static function streamExamples(): iterable
             )
         ];
 
+        yield AllFieldTypes::class . ' with JSON-type characters' => [
+            'data' => new AllFieldTypes(
+                string: '{ "with quotes", and { characters ".',
+            )
+        ];
+
         yield NullArrays::class => [
             'data' => new NullArrays(),
         ];