Skip to content

Commit

Permalink
bug #30773 [DependencyInjection] Fix hardcoded hotPathTagName (jderusse)
Browse files Browse the repository at this point in the history
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
  • Loading branch information
fabpot committed Mar 30, 2019
2 parents 6b8ff62 + 67fb8ef commit 1c92836
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -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 = [];
Expand Down

0 comments on commit 1c92836

Please sign in to comment.