minor #59203 [JsonEncoder] Reuse decodeString in NativeDecoder (alamirault)

chalasr · chalasr · commit d57efd1b041c · 2024-12-13T15:44:13.000+01:00
This PR was merged into the 7.3 branch. Discussion ---------- [JsonEncoder] Reuse decodeString in NativeDecoder | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | no | New feature? | no  | Deprecations? | no  | Issues | Fix #...  | License | MIT Commits ------- 0e83e1e [JsonEncoder] Use reuse decodeString in NativeDecoder
diff --git a/src/Symfony/Component/JsonEncoder/Decode/NativeDecoder.php b/src/Symfony/Component/JsonEncoder/Decode/NativeDecoder.php
@@ -40,10 +40,6 @@ public static function decodeStream($stream, int $offset = 0, ?int $length = nul
             $json = $stream->read($length);
         }
 
-        try {
-            return json_decode($json, associative: true, flags: \JSON_THROW_ON_ERROR);
-        } catch (\JsonException $e) {
-            throw new UnexpectedValueException('JSON is not valid: '.$e->getMessage());
-        }
+        return self::decodeString($json);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -40,10 +40,6 @@ public static function decodeStream($stream, int $offset = 0, ?int $length = nul`
`40`	`40`	`$json = $stream->read($length);`
`41`	`41`	`}`
`42`	`42`
`43`		`- try {`
`44`		`- return json_decode($json, associative: true, flags: \JSON_THROW_ON_ERROR);`
`45`		`- } catch (\JsonException $e) {`
`46`		`- throw new UnexpectedValueException('JSON is not valid: '.$e->getMessage());`
`47`		`- }`
	`43`	`+ return self::decodeString($json);`
`48`	`44`	`}`
`49`	`45`	`}`