Merge branch '5.4' into 6.0

derrabus · derrabus · commit e7d0d03ece08 · 2022-12-07T00:34:31.000+01:00
* 5.4: Revert "bug #48027 [DependencyInjection] Don't autoconfigure tag when it's already set with attributes (nicolas-grekas)"
diff --git a/src/Symfony/Component/DependencyInjection/Compiler/ResolveInstanceofConditionalsPass.php b/src/Symfony/Component/DependencyInjection/Compiler/ResolveInstanceofConditionalsPass.php
@@ -129,7 +129,7 @@ private function processDefinition(ContainerBuilder $container, string $id, Defi
                 foreach ($instanceofTags[$i] as $k => $v) {
                     if (null === $definition->getDecoratedService() || \in_array($k, $tagsToKeep, true)) {
                         foreach ($v as $v) {
-                            if ($definition->hasTag($k) && (!$v || \in_array($v, $definition->getTag($k)))) {
+                            if ($definition->hasTag($k) && \in_array($v, $definition->getTag($k))) {
                                 continue;
                             }
                             $definition->addTag($k, $v);
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Compiler/IntegrationTest.php b/src/Symfony/Component/DependencyInjection/Tests/Compiler/IntegrationTest.php
@@ -847,7 +847,6 @@ static function (ChildDefinition $definition, CustomAutoconfiguration $attribute
                 $definition->addTag('app.custom_tag', get_object_vars($attribute) + ['class' => $reflector->getName()]);
             }
         );
-        $container->registerForAutoconfiguration(TaggedService1::class)->addTag('app.custom_tag');
 
         $container->register('one', TaggedService1::class)
             ->setPublic(true)

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ private function processDefinition(ContainerBuilder $container, string $id, Defi`
`129`	`129`	`foreach ($instanceofTags[$i] as $k => $v) {`
`130`	`130`	`if (null === $definition->getDecoratedService() \|\| \in_array($k, $tagsToKeep, true)) {`
`131`	`131`	`foreach ($v as $v) {`
`132`		`- if ($definition->hasTag($k) && (!$v \|\| \in_array($v, $definition->getTag($k)))) {`
	`132`	`+ if ($definition->hasTag($k) && \in_array($v, $definition->getTag($k))) {`
`133`	`133`	`continue;`
`134`	`134`	`}`
`135`	`135`	`$definition->addTag($k, $v);`
Original file line number	Diff line number	Diff line change
`@@ -847,7 +847,6 @@ static function (ChildDefinition $definition, CustomAutoconfiguration $attribute`
`847`	`847`	`$definition->addTag('app.custom_tag', get_object_vars($attribute) + ['class' => $reflector->getName()]);`
`848`	`848`	`}`
`849`	`849`	`);`
`850`		`- $container->registerForAutoconfiguration(TaggedService1::class)->addTag('app.custom_tag');`
`851`	`850`
`852`	`851`	`$container->register('one', TaggedService1::class)`
`853`	`852`	`->setPublic(true)`