Merge pull request #68 from snapshotpl/remove-stdlib-dependency

Remove Zend\Stdlib dependency

Author: weierophinney

src/Config.php

@@ -9,25 +9,24 @@
 
 namespace Zend\ServiceManager;
 
-use Zend\Stdlib\ArrayUtils;
+use Zend\Stdlib\ArrayUtils\MergeRemoveKey;
+use Zend\Stdlib\ArrayUtils\MergeReplaceKeyInterface;
 
 class Config implements ConfigInterface
 {
     /**
-     * Allowed configuration keys
-     *
      * @var array
      */
-    protected $allowedKeys = [
+    private $allowedKeys = [
         'abstract_factories' => true,
-        'aliases'            => true,
-        'delegators'         => true,
-        'factories'          => true,
-        'initializers'       => true,
-        'invokables'         => true,
-        'lazy_services'      => true,
-        'services'           => true,
-        'shared'             => true,
+        'aliases' => true,
+        'delegators' => true,
+        'factories' => true,
+        'initializers' => true,
+        'invokables' => true,
+        'lazy_services' => true,
+        'services' => true,
+        'shared' => true,
     ];
 
     /**
@@ -46,8 +45,6 @@ class Config implements ConfigInterface
     ];
 
     /**
-     * Constructor
-     *
      * @param array $config
      */
     public function __construct(array $config = [])
@@ -58,15 +55,11 @@ public function __construct(array $config = [])
                 unset($config[$key]);
             }
         }
-
-        $this->config = ArrayUtils::merge($this->config, $config);
+        $this->config = $this->merge($this->config, $config);
     }
 
     /**
-     * Configure service manager
-     *
-     * @param ServiceManager $serviceManager
-     * @return ServiceManager Returns the updated service manager instance.
+     * @inheritdoc
      */
     public function configureServiceManager(ServiceManager $serviceManager)
     {
@@ -80,4 +73,34 @@ public function toArray()
     {
         return $this->config;
     }
+
+    /**
+     * Copy paste from https://github.com/zendframework/zend-stdlib/commit/26fcc32a358aa08de35625736095cb2fdaced090
+     * to keep compatibility with previous version
+     *
+     * @link https://github.com/zendframework/zend-servicemanager/pull/68
+     */
+    private function merge(array $a, array $b)
+    {
+        foreach ($b as $key => $value) {
+            if ($value instanceof MergeReplaceKeyInterface) {
+                $a[$key] = $value->getData();
+            } elseif (isset($a[$key]) || array_key_exists($key, $a)) {
+                if ($value instanceof MergeRemoveKey) {
+                    unset($a[$key]);
+                } elseif (is_int($key)) {
+                    $a[] = $value;
+                } elseif (is_array($value) && is_array($a[$key])) {
+                    $a[$key] = $this->merge($a[$key], $value);
+                } else {
+                    $a[$key] = $value;
+                }
+            } else {
+                if (!$value instanceof MergeRemoveKey) {
+                    $a[$key] = $value;
+                }
+            }
+        }
+        return $a;
+    }
 }

test/ConfigTest.php

@@ -18,6 +18,44 @@
  */
 class ConfigTest extends TestCase
 {
+    public function testMergeArrays()
+    {
+        $config = [
+            'invokables' => [
+                'foo' => TestAsset\InvokableObject::class,
+            ],
+            'delegators' => [
+                'foo' => [
+                    TestAsset\PreDelegator::class,
+                ]
+            ],
+            'factories' => [
+                'service' => TestAsset\FactoryObject::class,
+            ],
+        ];
+
+        $configuration = new TestAsset\ExtendedConfig($config);
+        $result = $configuration->toArray();
+
+        $expected = [
+            'invokables' => [
+                'foo' => TestAsset\InvokableObject::class,
+                TestAsset\InvokableObject::class => TestAsset\InvokableObject::class,
+            ],
+            'delegators' => [
+                'foo' => [
+                    TestAsset\InvokableObject::class,
+                    TestAsset\PreDelegator::class,
+                ],
+            ],
+            'factories' => [
+                'service' => TestAsset\FactoryObject::class,
+            ],
+        ];
+
+        $this->assertEquals($expected, $result);
+    }
+
     public function testPassesKnownServiceConfigKeysToServiceManagerWithConfigMethod()
     {
         $expected = [

test/TestAsset/ExtendedConfig.php

@@ -0,0 +1,26 @@
+<?php
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link      http://github.com/zendframework/zf2 for the canonical source repository
+ * @copyright Copyright (c) 2005-2015 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license   http://framework.zend.com/license/new-bsd New BSD License
+ */
+
+namespace ZendTest\ServiceManager\TestAsset;
+
+use Zend\ServiceManager\Config;
+
+class ExtendedConfig extends Config
+{
+    protected $config = [
+        'invokables' => [
+            InvokableObject::class => InvokableObject::class,
+        ],
+        'delegators' => [
+            'foo' => [
+                InvokableObject::class,
+            ],
+        ],
+    ];
+}