[JsonStreamer] Fix decoding iterable lists

mtarld · mtarld · commit 2b9b80134b3d · 2025-09-01T09:15:10.000+02:00
diff --git a/src/Symfony/Component/JsonStreamer/Read/PhpAstBuilder.php b/src/Symfony/Component/JsonStreamer/Read/PhpAstBuilder.php
@@ -379,10 +379,12 @@ private function buildCollectionNodeStatements(CollectionNode $node, bool $decod
             ? $this->builder->funcCall($this->builder->var('iterable'), [$this->builder->var('stream'), $this->builder->var('data')])
             : $this->builder->funcCall($this->builder->var('iterable'), [$this->builder->var('data')]);
 
+        $collectionKeyType = $node->getType()->getCollectionKeyType();
+
         $prepareDataStmts = $decodeFromStream ? [
             new Expression(new Assign($this->builder->var('data'), $this->builder->staticCall(
                 new FullyQualified(Splitter::class),
-                $node->getType()->isList() ? 'splitList' : 'splitDict',
+                ($collectionKeyType instanceof BuiltinType && TypeIdentifier::INT === $collectionKeyType->getTypeIdentifier()) ? 'splitList' : 'splitDict',
                 [$this->builder->var('stream'), $this->builder->var('offset'), $this->builder->var('length')],
             ))),
         ] : [];
diff --git a/src/Symfony/Component/JsonStreamer/Tests/JsonStreamReaderTest.php b/src/Symfony/Component/JsonStreamer/Tests/JsonStreamReaderTest.php
@@ -87,7 +87,7 @@ public function testReadCollection()
         $this->assertRead($reader, function (mixed $read) {
             $this->assertIsIterable($read);
             $this->assertSame([true, false], iterator_to_array($read));
-        }, '{"0": true, "1": false}', Type::iterable(Type::bool(), Type::int()));
+        }, '[true, false]', Type::iterable(Type::bool(), Type::int()));
     }
 
     public function testReadObject()

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ public function testReadCollection()`
`87`	`87`	`$this->assertRead($reader, function (mixed $read) {`
`88`	`88`	`$this->assertIsIterable($read);`
`89`	`89`	`$this->assertSame([true, false], iterator_to_array($read));`
`90`		`- }, '{"0": true, "1": false}', Type::iterable(Type::bool(), Type::int()));`
	`90`	`+ }, '[true, false]', Type::iterable(Type::bool(), Type::int()));`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`public function testReadObject()`