Updated ValueInjection to check array values as these can be exportable

CHANGELOG.md

@@ -22,7 +22,8 @@ All notable changes to this project will be documented in this file, in reverse
 
 ### Fixed
 
-- Nothing.
+- [#50](https://github.com/zendframework/zend-di/pull/50) fixes recognizing array values
+  as exportable, so factories can be generated for default array values.
 
 ## 3.1.0 - 2018-10-23
 

src/Resolver/ValueInjection.php

@@ -8,10 +8,16 @@
 namespace Zend\Di\Resolver;
 
 use Psr\Container\ContainerInterface;
-use ReflectionObject;
+use ReflectionMethod;
 use Zend\Di\Exception\LogicException;
 
+use function is_array;
+use function is_object;
+use function is_scalar;
+use function method_exists;
 use function trigger_error;
+use function var_export;
+
 use const E_USER_DEPRECATED;
 
 /**
@@ -68,15 +74,36 @@ public function export() : string
      */
     public function isExportable() : bool
     {
-        if (is_scalar($this->value) || ($this->value === null)) {
+        return $this->isExportableRecursive($this->value);
+    }
+
+    /**
+     * Check if the provided value is exportable.
+     * For arrays it uses recursion.
+     *
+     * @param mixed $value
+     * @return bool
+     */
+    private function isExportableRecursive($value) : bool
+    {
+        if (is_scalar($value) || $value === null) {
+            return true;
+        }
+
+        if (is_array($value)) {
+            foreach ($value as $item) {
+                if (! $this->isExportableRecursive($item)) {
+                    return false;
+                }
+            }
+
             return true;
         }
 
-        if (is_object($this->value) && method_exists($this->value, '__set_state')) {
-            $reflection = new ReflectionObject($this->value);
-            $method = $reflection->getMethod('__set_state');
+        if (is_object($value) && method_exists($value, '__set_state')) {
+            $method = new ReflectionMethod($value, '__set_state');
 
-            return ($method->isStatic() && $method->isPublic());
+            return $method->isStatic() && $method->isPublic();
         }
 
         return false;

test/Resolver/ValueInjectionTest.php

@@ -98,6 +98,23 @@ public function provideExportableValues()
             'null'         => [null],
             'float'        => [microtime(true)],
             'object'       => [new TestAsset\Resolver\ExportableValue()],
+            'array'        => [[]],
+            'array-string' => [['TestValue', 'OtherValue']],
+            'array-int'    => [[123, 456]],
+            'array-mixed'  => [
+                [
+                    new TestAsset\Resolver\ExportableValue(),
+                    [1],
+                    null,
+                    false,
+                    true,
+                    time(),
+                    microtime(true),
+                    [[], []],
+                    uniqid(),
+                    [],
+                ],
+            ],
         ];
     }
 
@@ -111,6 +128,8 @@ public function provideUnexportableItems()
             'stream'          => [$this->streamFixture],
             'noSetState'      => [new TestAsset\Resolver\UnexportableValue1()],
             'privateSetState' => [new TestAsset\Resolver\UnexportableValue2()],
+            'arrayNoSetState' => [[new TestAsset\Resolver\UnexportableValue1()]],
+            'arrayPrivateSetState' => [[new TestAsset\Resolver\UnexportableValue2()]],
         ];
     }