added attribute #Parameter

Author: dg

src/Application/Attributes/Parameter.php

@@ -0,0 +1,18 @@
+<?php
+
+/**
+ * This file is part of the Nette Framework (https://nette.org)
+ * Copyright (c) 2004 David Grudl (https://davidgrudl.com)
+ */
+
+declare(strict_types=1);
+
+namespace Nette\Application\Attributes;
+
+use Attribute;
+
+
+#[Attribute(Attribute::TARGET_PROPERTY)]
+class Parameter
+{
+}

src/Application/UI/Component.php

@@ -157,7 +157,7 @@ public static function getReflection(): ComponentReflection
 	public function loadState(array $params): void
 	{
 		$reflection = $this->getReflection();
-		foreach ($reflection->getPersistentParams() as $name => $meta) {
+		foreach ($reflection->getParameters() as $name => $meta) {
 			if (isset($params[$name])) { // nulls are ignored
 				if (!$reflection->convertType($params[$name], $meta['type'])) {
 					throw new Nette\Application\BadRequestException(sprintf(

src/Application/UI/ComponentReflection.php

@@ -33,9 +33,9 @@ final class ComponentReflection extends \ReflectionClass
 
 
 	/**
-	 * Returns array of persistent properties. They are public and have attribute #[Persistent] or annotation @persistent.
+	 * Returns array of class properties that are public and have attribute #[Persistent] or #[Parameter] or annotation @persistent.
 	 */
-	public function getPersistentParams(): array
+	public function getParameters(): array
 	{
 		$params = &self::$ppCache[$this->getName()];
 		if ($params !== null) {
@@ -46,26 +46,31 @@ public function getPersistentParams(): array
 		$isPresenter = $this->isSubclassOf(Presenter::class);
 		$defaults = $this->getDefaultProperties();
 		foreach ($this->getProperties(\ReflectionProperty::IS_PUBLIC) as $prop) {
-			if (!$prop->isStatic()
-				&& (self::parseAnnotation($prop, 'persistent')
-					|| (PHP_VERSION_ID >= 80000 && $prop->getAttributes(Nette\Application\Attributes\Persistent::class)))
+			if ($prop->isStatic()) {
+				continue;
+			} elseif (self::parseAnnotation($prop, 'persistent')
+				|| (PHP_VERSION_ID >= 80000 && $prop->getAttributes(Nette\Application\Attributes\Persistent::class))
 			) {
 				$default = $defaults[$prop->getName()] ?? null;
 				$params[$prop->getName()] = [
 					'def' => $default,
 					'type' => self::getPropertyType($prop, $default),
 					'since' => $isPresenter ? Nette\Utils\Reflection::getPropertyDeclaringClass($prop)->getName() : null,
 				];
+			} elseif (PHP_VERSION_ID >= 80000 && $prop->getAttributes(Nette\Application\Attributes\Parameter::class)) {
+				$params[$prop->getName()] = [
+					'type' => (string) ($prop->getType() ?? 'mixed'),
+				];
 			}
 		}
 
 		if ($this->getParentClass()->isSubclassOf(Component::class)) {
 			$parent = new self($this->getParentClass()->getName());
-			foreach ($parent->getPersistentParams() as $name => $meta) {
-				if (isset($params[$name])) {
-					$params[$name]['since'] = $meta['since'];
-				} else {
+			foreach ($parent->getParameters() as $name => $meta) {
+				if (!isset($params[$name])) {
 					$params[$name] = $meta;
+				} elseif (array_key_exists('since', $params[$name])) {
+					$params[$name]['since'] = $meta['since'];
 				}
 			}
 		}
@@ -74,6 +79,17 @@ public function getPersistentParams(): array
 	}
 
 
+	/**
+	 * Returns array of persistent properties. They are public and have attribute #[Persistent] or annotation @persistent.
+	 */
+	public function getPersistentParams(): array
+	{
+		return array_filter($this->getParameters(), function ($param) {
+			return array_key_exists('since', $param);
+		});
+	}
+
+
 	public function getPersistentComponents(): array
 	{
 		$class = $this->getName();
@@ -200,7 +216,7 @@ public static function convertType(&$val, string $types): bool
 		$scalars = ['string' => 1, 'int' => 1, 'float' => 1, 'bool' => 1, 'true' => 1, 'false' => 1, 'boolean' => 1, 'double' => 1, 'integer' => 1];
 		$testable = ['iterable' => 1, 'object' => 1, 'array' => 1, 'null' => 1];
 
-		foreach (explode('|', $types) as $type) {
+		foreach (explode('|', ltrim($types, '?')) as $type) {
 			if (isset($scalars[$type])) {
 				$ok = self::castScalar($val, $type);
 			} elseif (isset($testable[$type])) {

tests/UI/Presenter.getParameters.phpt

@@ -0,0 +1,107 @@
+<?php
+
+/**
+ * Test: Nette\Application\UI\Presenter::getRequestParams
+ */
+
+declare(strict_types=1);
+
+use Nette\Application\Attributes\Parameter;
+use Nette\Application\Attributes\Persistent;
+use Nette\Application\UI\ComponentReflection;
+use Nette\Application\UI\Presenter;
+use Tester\Assert;
+
+require __DIR__ . '/../bootstrap.php';
+
+
+class OnePresenter extends Presenter
+{
+	public static $no1;
+	public $no2;
+
+	/** @persistent */
+	public $yes1;
+
+	#[Persistent, Parameter]
+	public $yes2; /* Parameter is ignored */
+
+	#[Parameter]
+	public $yes3;
+}
+
+
+class TwoPresenter extends OnePresenter
+{
+	#[Parameter]
+	public $yes2;
+	public $yes3;
+
+	#[Parameter]
+	public $yes4;
+}
+
+
+if (PHP_VERSION_ID < 80000) {
+	Assert::same(
+		[
+			'yes1' => [
+				'def' => null,
+				'type' => 'scalar',
+				'since' => 'OnePresenter',
+			],
+		],
+		(new ComponentReflection(OnePresenter::class))->getParameters()
+	);
+
+	Assert::same(
+		[
+			'yes1' => [
+				'def' => null,
+				'type' => 'scalar',
+				'since' => 'OnePresenter',
+			],
+		],
+		(new ComponentReflection(TwoPresenter::class))->getParameters()
+	);
+
+} else {
+	Assert::same(
+		[
+			'yes1' => [
+				'def' => null,
+				'type' => 'scalar',
+				'since' => 'OnePresenter',
+			],
+			'yes2' => [
+				'def' => null,
+				'type' => 'scalar',
+				'since' => 'OnePresenter',
+			],
+			'yes3' => [
+				'type' => 'mixed',
+			],
+		],
+		(new ComponentReflection(OnePresenter::class))->getParameters()
+	);
+
+	Assert::same(
+		[
+			'yes2' => [
+				'type' => 'mixed',
+			],
+			'yes4' => [
+				'type' => 'mixed',
+			],
+			'yes1' => [
+				'def' => null,
+				'type' => 'scalar',
+				'since' => 'OnePresenter',
+			],
+			'yes3' => [
+				'type' => 'mixed',
+			],
+		],
+		(new ComponentReflection(TwoPresenter::class))->getParameters()
+	);
+}