Merge pull request ezsystems#1390 from ezsystems/configurationProcess…

…orPhp7 [PHP 7] Fixed reference issue in ConfigurationProcessor
ezpublishlegacy · Aug 31, 2015 · 968aa71 · 968aa71
2 parents cbb4894 + 393e75e
commit 968aa71
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...shCoreBundle/DependencyInjection/Configuration/SiteAccessAware/ConfigurationProcessor.php b/...shCoreBundle/DependencyInjection/Configuration/SiteAccessAware/ConfigurationProcessor.php
@@ -105,7 +105,7 @@ public function mapConfig(array $config, $mapper)
         $scopeNodeName = $this->contextualizer->getSiteAccessNodeName();
         foreach ($config[$scopeNodeName] as $currentScope => &$scopeSettings) {
             if ($mapperCallable) {
-                call_user_func_array($mapper, array($scopeSettings, $currentScope, $this->contextualizer));
+                call_user_func_array($mapper, array(&$scopeSettings, $currentScope, $this->contextualizer));
             } else {
                 $mapper->mapConfig($scopeSettings, $currentScope, $this->contextualizer);
             }

diff --git a/...le/Tests/DependencyInjection/Configuration/SiteAccessAware/ConfigurationProcessorTest.php b/...le/Tests/DependencyInjection/Configuration/SiteAccessAware/ConfigurationProcessorTest.php
@@ -101,7 +101,7 @@ public function testMapConfigClosure()
             ),
         );
 
-        $mapperClosure = function (array $scopeSettings, $currentScope, ContextualizerInterface $contextualizer) use ($config, $availableSAs, $saNodeName, $expectedContextualizer) {
+        $mapperClosure = function (array &$scopeSettings, $currentScope, ContextualizerInterface $contextualizer) use ($config, $availableSAs, $saNodeName, $expectedContextualizer) {
             self::assertTrue(isset($availableSAs[$currentScope]));
             self::assertTrue(isset($config[$saNodeName][$currentScope]));
             self::assertSame($config[$saNodeName][$currentScope], $scopeSettings);