[JsonStreamer] Fix encoding iterable lists

mtarld · mtarld · commit 2057bc8ac66a · 2025-09-01T20:34:39.000+02:00
diff --git a/src/Symfony/Component/JsonStreamer/Tests/JsonStreamWriterTest.php b/src/Symfony/Component/JsonStreamer/Tests/JsonStreamWriterTest.php
@@ -105,7 +105,7 @@ public function testWriteCollection()
         );
 
         $this->assertWritten(
-            '{"0":{"id":1,"name":"dummy"},"1":{"id":1,"name":"dummy"}}',
+            '[{"id":1,"name":"dummy"},{"id":1,"name":"dummy"}]',
             new \ArrayObject([new ClassicDummy(), new ClassicDummy()]),
             Type::iterable(Type::object(ClassicDummy::class), Type::int()),
         );
diff --git a/src/Symfony/Component/JsonStreamer/Write/PhpAstBuilder.php b/src/Symfony/Component/JsonStreamer/Write/PhpAstBuilder.php
@@ -294,7 +294,9 @@ private function buildYieldStatements(DataModelNodeInterface $dataModelNode, arr
         if ($dataModelNode instanceof CollectionNode) {
             ++$context['depth'];
 
-            if ($dataModelNode->getType()->isList()) {
+            $collectionKeyType = $dataModelNode->getType()->getCollectionKeyType();
+
+            if ($collectionKeyType instanceof BuiltinType && TypeIdentifier::INT === $collectionKeyType->getTypeIdentifier()) {
                 return [
                     new Expression(new Yield_($this->builder->val('['))),
                     new Expression(new Assign($this->builder->var('prefix'), $this->builder->val(''))),
