Merge branch 'hotfix/3070'

Close zendframework/zendframework#3070

src/Definition/RuntimeDefinition.php

@@ -346,7 +346,7 @@ protected function processParams(&$def, Reflection\ClassReflection $rClass, Refl
             // set the class name, if it exists
             $def['parameters'][$methodName][$fqName][] = $actualParamName;
             $def['parameters'][$methodName][$fqName][] = ($p->getClass() !== null) ? $p->getClass()->getName() : null;
-            $def['parameters'][$methodName][$fqName][] = !($optional =$p->isOptional());
+            $def['parameters'][$methodName][$fqName][] = !($optional =$p->isOptional() && $p->isDefaultValueAvailable());
             $def['parameters'][$methodName][$fqName][] = $optional ? $p->getDefaultValue() : null;
         }
 

test/Definition/RuntimeDefinitionTest.php

@@ -59,4 +59,38 @@ public function testIncludesDefaultMethodParameters()
             )
         );
     }
+
+    public function testExceptionDefaultValue()
+    {
+        $definition = new RuntimeDefinition();
+
+        $definition->forceLoadClass('RecursiveIteratorIterator');
+
+        $this->assertSame(
+            array(
+                'RecursiveIteratorIterator::__construct:0' => array(
+                    'iterator',
+                    'Traversable',
+                    true,
+                    null,
+                ),
+                'RecursiveIteratorIterator::__construct:1' => Array (
+                    'mode',
+                    null,
+                    true,
+                    null,
+                ),
+                'RecursiveIteratorIterator::__construct:2' => Array (
+                    'flags',
+                    null,
+                    true,
+                    null,
+                ),
+            ),
+            $definition->getMethodParameters(
+                'RecursiveIteratorIterator',
+                '__construct'
+            )
+        );
+    }
 }