LinkGenerator: do not pass defaults of #[Parameter] (#355)

Author: vrana

src/Application/LinkGenerator.php

@@ -170,6 +170,12 @@ public function createRequest(
 				trigger_error("Link to deprecated presenter '$presenter' from '{$refPresenter->getName()}:{$refPresenter->getAction()}'.", E_USER_DEPRECATED);
 			}
 
+			foreach (array_intersect_key($reflection->getParameters(), $args) as $name => $param) {
+				if ($args[$name] === $param['def']) {
+					$args[$name] = null; // value transmit is unnecessary
+				}
+			}
+
 			// counterpart of run() & tryCall()
 			if ($method = $reflection->getActionRenderMethod($action)) {
 				if ($this->isDeprecated($refPresenter, $method)) {

src/Application/UI/ComponentReflection.php

@@ -55,6 +55,7 @@ public function getParameters(): array
 				];
 			} elseif ($prop->getAttributes(Attributes\Parameter::class)) {
 				$params[$prop->getName()] = [
+					'def' => $prop->hasDefaultValue() ? $prop->getDefaultValue() : null,
 					'type' => (string) ($prop->getType() ?? 'mixed'),
 				];
 			}

tests/UI/ComponentReflection.getParameters().phpt

@@ -51,6 +51,7 @@ Assert::same(
 			'since' => 'OnePresenter',
 		],
 		'yes3' => [
+			'def' => null,
 			'type' => 'mixed',
 		],
 	],
@@ -60,9 +61,11 @@ Assert::same(
 Assert::same(
 	[
 		'yes2' => [
+			'def' => null,
 			'type' => 'mixed',
 		],
 		'yes4' => [
+			'def' => null,
 			'type' => 'mixed',
 		],
 		'yes1' => [
@@ -71,6 +74,7 @@ Assert::same(
 			'since' => 'OnePresenter',
 		],
 		'yes3' => [
+			'def' => null,
 			'type' => 'mixed',
 		],
 	],