MicroPresenter: removal of callback from parameters during canonizati…

…on [Closes nette/routing#9]
nette · Feb 8, 2021 · f57d5cc · f57d5cc
1 parent 68e1ffa
commit f57d5cc
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/src/Application/MicroPresenter.php b/src/Application/MicroPresenter.php
@@ -61,24 +61,27 @@ public function run(Application\Request $request): Application\Response
 	{
 		$this->request = $request;
 
+		$params = $request->getParameters();
+		$callback = $params['callback'] ?? null;
+		unset($params['callback']);
+
+		if (!is_object($callback) || !is_callable($callback)) {
+			throw new Application\BadRequestException('Parameter callback is not a valid closure.');
+		}
+
 		if (
 			$this->httpRequest
 			&& $this->router
 			&& !$this->httpRequest->isAjax()
 			&& ($request->isMethod('get') || $request->isMethod('head'))
 		) {
 			$refUrl = $this->httpRequest->getUrl()->withoutUserInfo();
-			$url = $this->router->constructUrl($request->toArray(), $refUrl);
+			$url = $this->router->constructUrl($params, $refUrl);
 			if ($url !== null && !$refUrl->isEqual($url)) {
 				return new Responses\RedirectResponse($url, Http\IResponse::S301_MOVED_PERMANENTLY);
 			}
 		}
 
-		$params = $request->getParameters();
-		$callback = $params['callback'] ?? null;
-		if (!is_object($callback) || !is_callable($callback)) {
-			throw new Application\BadRequestException('Parameter callback is not a valid closure.');
-		}
 		$reflection = Nette\Utils\Callback::toReflection($callback);
 
 		if ($this->context) {