[DependencyInjection] Ensure Definition::getTags() contains a list of attributes for each tag

GromNaN · GromNaN · commit b3c474eee429 · 2025-12-07T21:47:05.000+01:00
diff --git a/src/Symfony/Component/DependencyInjection/Definition.php b/src/Symfony/Component/DependencyInjection/Definition.php
@@ -426,7 +426,7 @@ public function isAutoconfigured(): bool
      */
     public function setTags(array $tags): static
     {
-        $this->tags = $tags;
+        $this->tags = array_map('array_values', $tags);
 
         return $this;
     }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/DefinitionTest.php b/src/Symfony/Component/DependencyInjection/Tests/DefinitionTest.php
@@ -216,6 +216,19 @@ public function testSetGetConfigurator()
         $this->assertEquals('foo', $def->getConfigurator(), '->getConfigurator() returns the configurator');
     }
 
+    public function testSetTagsAreInList()
+    {
+        $def = new Definition('stdClass');
+        $def->setTags([
+            'tag1' => [5 => ['attr1' => 'value1'], 3 => []],
+            'tag2' => [1 => ['attr2' => 'value2']],
+        ]);
+        $this->assertSame([
+            'tag1' => [['attr1' => 'value1'], []],
+            'tag2' => [['attr2' => 'value2']],
+        ], $def->getTags(), '->setTags() reindex the tags arrays');
+    }
+
     public function testClearTags()
     {
         $def = new Definition('stdClass');

Original file line number	Diff line number	Diff line change
`@@ -426,7 +426,7 @@ public function isAutoconfigured(): bool`
`426`	`426`	`*/`
`427`	`427`	`public function setTags(array $tags): static`
`428`	`428`	`{`
`429`		`- $this->tags = $tags;`
	`429`	`+ $this->tags = array_map('array_values', $tags);`
`430`	`430`
`431`	`431`	`return $this;`
`432`	`432`	`}`