Component: createComponent<name> methods are checked by checkRequirements()

Author: dg

src/Application/UI/Component.php

@@ -73,6 +73,9 @@ public function getUniqueId(): string
 
 	protected function createComponent(string $name): ?Nette\ComponentModel\IComponent
 	{
+		if (method_exists($this, $method = 'createComponent' . $name)) {
+			$this->checkRequirements(new \ReflectionMethod($this, $method));
+		}
 		$res = parent::createComponent($name);
 		if ($res && !$res instanceof SignalReceiver && !$res instanceof StatePersistent) {
 			$type = $res::class;

tests/UI/Requires.locations.phpt

@@ -52,6 +52,38 @@ class TestMethodHandlePresenter extends Nette\Application\UI\Presenter
 }
 
 
+class TestMethodCreateComponentPresenter extends Nette\Application\UI\Presenter
+{
+	#[Requires(forward: true)]
+	public function createComponentFoo(): never
+	{
+		$this->terminate();
+	}
+
+
+	public function createComponentBar(): TestMethodCreateComponentControl
+	{
+		return new TestMethodCreateComponentControl;
+	}
+}
+
+class TestMethodCreateComponentControl extends Nette\Application\UI\Control
+{
+	#[Requires(forward: true)]
+	public function createComponentFoo(): never
+	{
+		$this->getPresenter()->terminate();
+	}
+
+
+	#[Requires(actions: 'foo')]
+	public function createComponentBad(): void
+	{
+	}
+}
+
+
+
 // class-level attribute
 $presenter = createPresenter(TestClassPresenter::class);
 Assert::noError(
@@ -102,3 +134,38 @@ Assert::exception(
 	Application\BadRequestException::class,
 	'Forwarded request is required by TestMethodHandlePresenter::handleFoo()',
 );
+
+
+// method createComponent<name>()
+$presenter = createPresenter(TestMethodCreateComponentPresenter::class);
+Assert::noError(
+	fn() => $presenter->run(new Application\Request('', Application\Request::FORWARD, ['foo-a' => 1])),
+);
+
+Assert::exception(
+	fn() => $presenter->run(new Application\Request('', Http\Request::Get, ['foo-a' => 1])),
+	Application\BadRequestException::class,
+	'Forwarded request is required by TestMethodCreateComponentPresenter::createComponentFoo()',
+);
+
+
+// method createComponent<name>() in component
+$presenter = createPresenter(TestMethodCreateComponentPresenter::class);
+Assert::noError(
+	fn() => $presenter->run(new Application\Request('', Application\Request::FORWARD, ['bar-foo-a' => 1])),
+);
+
+Assert::exception(
+	fn() => $presenter->run(new Application\Request('', Http\Request::Get, ['bar-foo-a' => 1])),
+	Application\BadRequestException::class,
+	'Forwarded request is required by TestMethodCreateComponentControl::createComponentFoo()',
+);
+
+
+// option actions in method createComponent<name>() in component
+$presenter = createPresenter(TestMethodCreateComponentPresenter::class);
+Assert::exception(
+	fn() => $presenter->run(new Application\Request('', Http\Request::Get, ['bar-bad-a' => 1])),
+	Nette\InvalidStateException::class,
+	"Option 'actions' used by TestMethodCreateComponentControl::createComponentBad() is allowed only in presenter.",
+);