Component: only UI components can be added to presenter/component (BC break) WIP

Author: dg

src/Application/UI/Component.php

@@ -71,20 +71,28 @@ public function getUniqueId(): string
 	}
 
 
+	public function addComponent(
+		Nette\ComponentModel\IComponent $component,
+		?string $name,
+		?string $insertBefore = null,
+	): static
+	{
+		if (!$component instanceof SignalReceiver && !$component instanceof StatePersistent) {
+			throw new Nette\InvalidStateException("Component '$name' of type " . get_debug_type($component) . ' is not intended to be used in the Presenter.');
+		}
+
+		return parent::addComponent($component, $name, $insertBefore = null);
+	}
+
+
 	protected function createComponent(string $name): ?Nette\ComponentModel\IComponent
 	{
 		if (method_exists($this, $method = 'createComponent' . $name)) {
 			$rm = new \ReflectionMethod($this, $method);
 			static::getReflection()->checkRequirements($rm, $this->getPresenterIfExists());
 			$this->checkRequirements($rm);
 		}
-		$res = parent::createComponent($name);
-		if ($res && !$res instanceof SignalReceiver && !$res instanceof StatePersistent) {
-			$type = $res::class;
-			trigger_error("It seems that component '$name' of type $type is not intended to be used in the Presenter.");
-		}
-
-		return $res;
+		return parent::createComponent($name);
 	}