bug #30773 [DependencyInjection] Fix hardcoded hotPathTagName (jderusse)

fabpot · fabpot · commit 1c92836e8c77 · 2019-03-30T08:04:10.000+01:00
This PR was merged into the 3.4 branch. Discussion ---------- [DependencyInjection] Fix hardcoded hotPathTagName | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | none | License | MIT | Doc PR | none replace the hardcoded string by the injected variable Commits ------- 67fb8ef Fix hardcoded hotPathTagName
diff --git a/src/Symfony/Component/EventDispatcher/DependencyInjection/RegisterListenersPass.php b/src/Symfony/Component/EventDispatcher/DependencyInjection/RegisterListenersPass.php
@@ -106,7 +106,7 @@ public function process(ContainerBuilder $container)
                 $definition->addMethodCall('addListener', $args);
 
                 if (isset($this->hotPathEvents[$args[0]])) {
-                    $container->getDefinition($id)->addTag('container.hot_path');
+                    $container->getDefinition($id)->addTag($this->hotPathTagName);
                 }
             }
             $extractingDispatcher->listeners = [];

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ public function process(ContainerBuilder $container)`
`106`	`106`	`$definition->addMethodCall('addListener', $args);`
`107`	`107< 5C53 /code>`
`108`	`108`	`if (isset($this->hotPathEvents[$args[0]])) {`
`109`		`- $container->getDefinition($id)->addTag('container.hot_path');`
	`109`	`+ $container->getDefinition($id)->addTag($this->hotPathTagName);`
`110`	`110`	`}`
`111`	`111`	`}`
`112`	`112`	`$extractingDispatcher->listeners = [];`