Skip to content

Commit a6f01a3

Browse files
committed
minor symfony#24249 [DI] remove confusing code (nicolas-grekas)
This PR was merged into the 2.7 branch. Discussion ---------- [DI] remove confusing code | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - This code is a no-op. I guess the intention was to resolve a parametrized alias target, yet this never worked and we don't want to make it work. Commits ------- 3da645e [DI] remove confusing code
2 parents c3518b5 + 3da645e commit a6f01a3

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Symfony/Component/DependencyInjection/Compiler/ResolveParameterPlaceHoldersPass.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ public function process(ContainerBuilder $container)
6464

6565
$aliases = array();
6666
foreach ($container->getAliases() as $name => $target) {
67-
$aliases[$parameterBag->resolveValue($name)] = $parameterBag->resolveValue($target);
67+
$aliases[$parameterBag->resolveValue($name)] = $target;
6868
}
6969
$container->setAliases($aliases);
7070

0 commit comments

Comments
 (0)