Merge pull request #16 from microsoft/dev

Release 0.4.1
microsoft · Mar 6, 2023 · 81a18e3 · 81a18e3
2 parents 9044dd6 + 042b711
commit 81a18e3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/JsonSerializationWriter.php b/src/JsonSerializationWriter.php
@@ -45,7 +45,7 @@ private function writePropertyName(string $propertyName): void {
      */
     public function writeStringValue(?string $key, ?string $value): void {
 
-        $propertyValue = $value !== null ? '"'.addcslashes($value, "\\\r\n\"").'"' : '';
+        $propertyValue = $value !== null ? '"'.addcslashes($value, "\\\r\n\"\t").'"' : '';
         if ($value !== null) {
             if (!empty($key)) {
                 $this->writePropertyName($key);

diff --git a/tests/JsonSerializationWriterTest.php b/tests/JsonSerializationWriterTest.php
@@ -201,8 +201,8 @@ public function testGetSerializedContent(): void{
      */
     public function testWriteStringValue(): void {
         $this->jsonSerializationWriter = new JsonSerializationWriter();
-        $this->jsonSerializationWriter->writeAnyValue("statement", "This is a string");
-        $expected = '"statement":"This is a string"';
+        $this->jsonSerializationWriter->writeAnyValue("statement", "This is a string\n\r\t");
+        $expected = '"statement":"This is a string\\n\\r\\t"';
         $actual = $this->jsonSerializationWriter->getSerializedContent()->getContents();
         $this->assertEquals($expected, $actual);
     }