Merge pull request #14 from Majkl578/php56/remove-compat-checks

dg · dg · commit 38e8fbb57871 · 2015-08-26T13:44:21.000+02:00
Remove PHP &lt;=5.6 compatibility checks
diff --git a/src/PhpGenerator/Parameter.php b/src/PhpGenerator/Parameter.php
@@ -55,7 +55,7 @@ public static function from(\ReflectionParameter $from)
 				}
 			}
 		}
-		$param->optional = PHP_VERSION_ID < 50407 ? $from->isOptional() || ($param->typeHint && $from->allowsNull()) : $from->isDefaultValueAvailable();
+		$param->optional = $from->isDefaultValueAvailable();
 		$param->defaultValue = $from->isDefaultValueAvailable() ? $from->getDefaultValue() : NULL;
 
 		$namespace = $from->getDeclaringClass() ? $from->getDeclaringClass()->getNamespaceName() : NULL;

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public static function from(\ReflectionParameter $from)`
`55`	`55`	`}`
`56`	`56`	`}`
`57`	`57`	`}`
`58`		`- $param->optional = PHP_VERSION_ID < 50407 ? $from->isOptional() \|\| ($param->typeHint && $from->allowsNull()) : $from->isDefaultValueAvailable();`
	`58`	`+ $param->optional = $from->isDefaultValueAvailable();`
`59`	`59`	`$param->defaultValue = $from->isDefaultValueAvailable() ? $from->getDefaultValue() : NULL;`
`60`	`60`
`61`	`61`	`$namespace = $from->getDeclaringClass() ? $from->getDeclaringClass()->getNamespaceName() : NULL;`