PhpReflection::getParameterType() can handle PHP 7.1 nullable types (#126)

Author: MartinMajor

src/DI/PhpReflection.php

@@ -59,8 +59,10 @@ public static function getDeclaringClass(\ReflectionProperty $prop)
 	public static function getParameterType(\ReflectionParameter $param)
 	{
 		if (PHP_VERSION_ID >= 70000) {
-			$type = $param->hasType() ? (string) $param->getType() : NULL;
-			return strtolower($type) === 'self' ? $param->getDeclaringClass()->getName() : $type;
+			if ($param->hasType()) {
+				$type = PHP_VERSION_ID >= 70100 ? $param->getType()->getName() : (string) $param->getType();
+				return strtolower($type) === 'self' ? $param->getDeclaringClass()->getName() : $type;
+			}
 		} elseif ($param->isArray() || $param->isCallable()) {
 			return $param->isArray() ? 'array' : 'callable';
 		} else {

tests/DI/PhpReflection.getParameterType.php71.phpt

@@ -0,0 +1,31 @@
+<?php
+
+/**
+ * Test: Nette\DI\PhpReflection::getParameterType
+ * @phpversion 7.1
+ */
+
+use Nette\DI\PhpReflection;
+use Tester\Assert;
+
+
+require __DIR__ . '/../bootstrap.php';
+
+
+use Test\B; // for testing purposes
+
+class A
+{
+	function method(Undeclared $undeclared, B $b, array $array, callable $callable, $none, ?B $nullable)
+	{}
+}
+
+$method = new \ReflectionMethod('A', 'method');
+$params = $method->getParameters();
+
+Assert::same('Undeclared', PhpReflection::getParameterType($params[0]));
+Assert::same('Test\B', PhpReflection::getParameterType($params[1]));
+Assert::same('array', PhpReflection::getParameterType($params[2]));
+Assert::same('callable', PhpReflection::getParameterType($params[3]));
+Assert::null(PhpReflection::getParameterType($params[4]));
+Assert::same('Test\B', PhpReflection::getParameterType($params[5]));