PresenterComponentReflection::combineArgs() should retype variables i…

…n the $res array. [Fixes #99]

Use foreach's automatic indexing instead of manual one.

src/Application/UI/PresenterComponentReflection.php

@@ -117,15 +117,14 @@ public function hasCallableMethod($method)
 	public static function combineArgs(\ReflectionFunctionAbstract $method, $args)
 	{
 		$res = array();
-		$i = 0;
-		foreach ($method->getParameters() as $param) {
+		foreach ($method->getParameters() as $i => $param) {
 			$name = $param->getName();
 			if (!isset($args[$name]) && $param->isDefaultValueAvailable()) {
-				$res[$i++] = $param->getDefaultValue();
+				$res[$i] = $param->getDefaultValue();
 			} else {
-				$res[$i++] = $arg = isset($args[$name]) ? $args[$name] : NULL;
+				$res[$i] = $arg = isset($args[$name]) ? $args[$name] : NULL;
 				list($type, $isClass) = self::getParameterType($param);
-				if (!self::convertType($arg, $type, $isClass)) {
+				if (!self::convertType($res[$i], $type, $isClass)) {
 					throw new BadRequestException(sprintf(
 						'Argument $%s passed to %s() must be %s, %s given.',
 						$name,

tests/UI/PresenterComponentReflection.combineArgs.phpt

@@ -0,0 +1,24 @@
+<?php
+
+/**
+ * Test: PresenterComponentReflection::combineArgs()
+ */
+
+use Nette\Application\UI\PresenterComponentReflection;
+use Tester\Assert;
+
+require __DIR__ . '/../bootstrap.php';
+
+class MyPresenter
+{
+
+	public function myMethod($intParam = 0, $strParam = '')
+	{
+	}
+
+}
+
+$reflection = new ReflectionMethod('MyPresenter', 'myMethod');
+
+Assert::same(array(10, 'str'), PresenterComponentReflection::combineArgs($reflection, array('intParam' => '10', 'strParam' => 'str')));
+Assert::same(array(0, ''), PresenterComponentReflection::combineArgs($reflection, array()));