[DI] accept null index in #[TaggedItem]

nicolas-grekas · nicolas-grekas · commit 60518072541e · 2021-03-16T12:16:39.000+01:00
diff --git a/src/Symfony/Component/DependencyInjection/Attribute/TaggedItem.php b/src/Symfony/Component/DependencyInjection/Attribute/TaggedItem.php
@@ -20,7 +20,7 @@
 class TaggedItem
 {
     public function __construct(
-        public string $index,
+        public ?string $index = null,
         public ?int $priority = null,
     ) {
     }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Compiler/PriorityTaggedServiceTraitTest.php b/src/Symfony/Component/DependencyInjection/Tests/Compiler/PriorityTaggedServiceTraitTest.php
@@ -205,13 +205,17 @@ public function testTaggedItemAttributes()
                 HelloNamedService::class => (new ChildDefinition(''))->addTag('my_custom_tag'),
                 \stdClass::class => (new ChildDefinition(''))->addTag('my_custom_tag2'),
             ]);
+        $container->register('service3', HelloNamedService2::class)
+            ->setAutoconfigured(true)
+            ->addTag('my_custom_tag');
 
         (new ResolveInstanceofConditionalsPass())->process($container);
 
         $priorityTaggedServiceTraitImplementation = new PriorityTaggedServiceTraitImplementation();
 
         $tag = new TaggedIteratorArgument('my_custom_tag', 'foo', 'getFooBar');
         $expected = [
+            'service3' => new TypedReference('service3', HelloNamedService2::class),
             'hello' => new TypedReference('service2', HelloNamedService::class),
             'service1' => new TypedReference('service1', FooTagClass::class),
         ];
@@ -231,7 +235,13 @@ public function test($tagName, ContainerBuilder $container)
     }
 }
 
+
 #[TaggedItem(index: 'hello', priority: 1)]
 class HelloNamedService extends \stdClass
 {
 }
+
+#[TaggedItem(priority: 2)]
+class HelloNamedService2
+{
+}

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@`
`20`	`20`	`class TaggedItem`
`21`	`21`	`{`
`22`	`22`	`public function __construct(`
`23`		`- public string $index,`
	`23`	`+ public ?string $index = null,`
`24`	`24`	`public ?int $priority = null,`
`25`	`25`	`) {`
`26`	`26`	`}`