[DI] Fix keys resolution in ResolveParameterPlaceHoldersPass

nicolas-grekas · nicolas-grekas · commit 4d9d92afa0c6 · 2017-06-12T12:05:03.000+02:00
diff --git a/src/Symfony/Component/DependencyInjection/Compiler/ResolveParameterPlaceHoldersPass.php b/src/Symfony/Component/DependencyInjection/Compiler/ResolveParameterPlaceHoldersPass.php
@@ -65,10 +65,11 @@ protected function processValue($value, $isRoot = false)
             if (isset($changes['file'])) {
                 $value->setFile($this->bag->resolveValue($value->getFile()));
             }
-            $value->setProperties($this->bag->resolveValue($value->getProperties()));
-            $value->setMethodCalls($this->bag->resolveValue($value->getMethodCalls()));
+        }
+        if (is_array($value = parent::processValue($value, $isRoot)) && $value) {
+            $value = array_combine($this->bag->resolveValue(array_keys($value)), $value);
         }
 
-        return parent::processValue($value, $isRoot);
+        return $value;
     }
 }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Compiler/ResolveParameterPlaceHoldersPassTest.php b/src/Symfony/Component/DependencyInjection/Tests/Compiler/ResolveParameterPlaceHoldersPassTest.php
@@ -41,12 +41,12 @@ public function testFactoryParametersShouldBeResolved()
 
     public function testArgumentParametersShouldBeResolved()
     {
-        $this->assertSame(array('bar', 'baz'), $this->fooDefinition->getArguments());
+        $this->assertSame(array('bar', array('bar' => 'baz')), $this->fooDefinition->getArguments());
     }
 
     public function testMethodCallParametersShouldBeResolved()
     {
-        $this->assertSame(array(array('foobar', array('bar', 'baz'))), $this->fooDefinition->getMethodCalls());
+        $this->assertSame(array(array('foobar', array('bar', array('bar' => 'baz')))), $this->fooDefinition->getMethodCalls());
     }
 
     public function testPropertyParametersShouldBeResolved()
@@ -71,7 +71,7 @@ private function createContainerBuilder()
         $containerBuilder->setParameter('foo.class', 'Foo');
         $containerBuilder->setParameter('foo.factory.class', 'FooFactory');
         $containerBuilder->setParameter('foo.arg1', 'bar');
-        $containerBuilder->setParameter('foo.arg2', 'baz');
+        $containerBuilder->setParameter('foo.arg2', array('%foo.arg1%' => 'baz'));
         $containerBuilder->setParameter('foo.method', 'foobar');
         $containerBuilder->setParameter('foo.property.name', 'bar');
         $containerBuilder->setParameter('foo.property.value', 'baz');
@@ -80,7 +80,7 @@ private function createContainerBuilder()
 
         $fooDefinition = $containerBuilder->register('foo', '%foo.class%');
         $fooDefinition->setFactory(array('%foo.factory.class%', 'getFoo'));
-        $fooDefinition->setArguments(array('%foo.arg1%', '%foo.arg2%'));
+        $fooDefinition->setArguments(array('%foo.arg1%', array('%foo.arg1%' => 'baz')));
         $fooDefinition->addMethodCall('%foo.method%', array('%foo.arg1%', '%foo.arg2%'));
         $fooDefinition->setProperty('%foo.property.name%', '%foo.property.value%');
         $fooDefinition->setFile('%foo.file%');

Original file line number	Diff line number	Diff line change
`@@ -65,10 +65,11 @@ protected function processValue($value, $isRoot = false)`
`65`	`65`	`if (isset($changes['file'])) {`
`66`	`66`	`$value->setFile($this->bag->resolveValue($value->getFile()));`
`67`	`67`	`}`
`68`		`- $value->setProperties($this->bag->resolveValue($value->getProperties()));`
`69`		`- $value->setMethodCalls($this->bag->resolveValue($value->getMethodCalls()));`
	`68`	`+ }`
	`69`	`+ if (is_array($value = parent::processValue($value, $isRoot)) && $value) {`
	`70`	`+ $value = array_combine($this->bag->resolveValue(array_keys($value)), $value);`
`70`	`71`	`}`
`71`	`72`
`72`		`- return parent::processValue($value, $isRoot);`
	`73`	`+ return $value;`
`73`	`74`	`}`
`74`	`75`	`}`