removed useless type juggling and checking

Author: dg

src/Application/Application.php

@@ -105,7 +105,7 @@ public function run(): void
 	public function createInitialRequest(): Request
 	{
 		$request = $this->router->match($this->httpRequest);
-		if (!$request instanceof Request) {
+		if (!$request) {
 			throw new BadRequestException('No route for HTTP request.');
 		}
 		return $request;

src/Application/PresenterFactory.php

@@ -63,7 +63,7 @@ public function getPresenterClass(string &$name): string
 			return $this->cache[$name];
 		}
 
-		if (!is_string($name) || !Nette\Utils\Strings::match($name, '#^[a-zA-Z\x7f-\xff][a-zA-Z0-9\x7f-\xff:]*\z#')) {
+		if (!Nette\Utils\Strings::match($name, '#^[a-zA-Z\x7f-\xff][a-zA-Z0-9\x7f-\xff:]*\z#')) {
 			throw new InvalidPresenterException("Presenter name must be alphanumeric string, '$name' is invalid.");
 		}
 

src/Application/Request.php

@@ -199,7 +199,7 @@ public function isMethod(string $method): bool
 	 */
 	public function setFlag(string $flag, bool $value = TRUE)
 	{
-		$this->flags[$flag] = (bool) $value;
+		$this->flags[$flag] = $value;
 		return $this;
 	}
 

src/Application/Responses/RedirectResponse.php

@@ -29,8 +29,8 @@ class RedirectResponse implements Nette\Application\IResponse
 
 	public function __construct(string $url, int $httpCode = Http\IResponse::S302_FOUND)
 	{
-		$this->url = (string) $url;
-		$this->httpCode = (int) $httpCode;
+		$this->url = $url;
+		$this->httpCode = $httpCode;
 	}
 
 

src/Application/UI/Component.php

@@ -78,7 +78,7 @@ protected function validateParent(Nette\ComponentModel\IContainer $parent): void
 	protected function tryCall(string $method, array $params): bool
 	{
 		$rc = $this->getReflection();
-		if ($rc->hasMethod((string) $method)) {
+		if ($rc->hasMethod($method)) {
 			$rm = $rc->getMethod($method);
 			if ($rm->isPublic() && !$rm->isAbstract() && !$rm->isStatic()) {
 				$this->checkRequirements($rm);
@@ -254,7 +254,7 @@ public function signalReceived(string $signal): void
 	 */
 	public static function formatSignalMethod(string $signal): string
 	{
-		return $signal == NULL ? NULL : 'handle' . $signal; // intentionally ==
+		return 'handle' . $signal;
 	}
 
 

src/Application/UI/Control.php

@@ -45,12 +45,7 @@ public function setTemplateFactory(ITemplateFactory $templateFactory)
 	public function getTemplate(): ITemplate
 	{
 		if ($this->template === NULL) {
-			$value = $this->createTemplate();
-			if (!$value instanceof ITemplate && $value !== NULL) {
-				$class2 = get_class($value); $class = get_class($this);
-				throw new Nette\UnexpectedValueException("Object returned by $class::createTemplate() must be instance of Nette\\Application\\UI\\ITemplate, '$class2' given.");
-			}
-			$this->template = $value;
+			$this->template = $this->createTemplate();
 		}
 		return $this->template;
 	}

src/Application/UI/Presenter.php

@@ -371,7 +371,7 @@ public function getAction(bool $fullyQualified = FALSE): string
 	 */
 	public function changeAction(string $action): void
 	{
-		$this->action = $this->view = (string) $action;
+		$this->action = $this->view = $action;
 	}
 
 
@@ -390,7 +390,7 @@ public function getView(): string
 	 */
 	public function setView(string $view)
 	{
-		$this->view = (string) $view;
+		$this->view = $view;
 		return $this;
 	}
 
@@ -623,7 +623,7 @@ public function forward($destination, $args = []): void
 	public function redirectUrl(string $url, int $httpCode = NULL): void
 	{
 		if ($this->isAjax()) {
-			$this->payload->redirect = (string) $url;
+			$this->payload->redirect = $url;
 			$this->sendPayload();
 
 		} elseif (!$httpCode) {