SimpleRouter – match()

Upozornění: Tohle vlákno je hodně staré a informace nemusí být platné pro současné Nette.
pheidrich
Člen | 11
+
0
-

Nevím, jestli to může mít nějaký katastrofický dopad :), ale každopádně jsem narazil v SimpleRouteru na potenciálně nedefinovanou proměnnou: $presenter v metodě match(), řádek 88.

	public function match(/*Nette\Web\*/IHttpRequest $context)
	{
		// combine with precedence: get, (post,) defaults
		$params = $context->getQuery();
		$params += $this->defaults;

		if (isset($params[self::PRESENTER_KEY])) {
			$presenter = $this->module . $params[self::PRESENTER_KEY];
			unset($params[self::PRESENTER_KEY]);
		}

		return new PresenterRequest(
			$presenter,
			$context->getMethod(),
			$params,
			$context->getPost(),
			$context->getFiles(),
			array('secured' => $context->isSecured())
		);
	}

Pokud vytvořím SimpleRouter s prázdnými $defaults parametry a předám mu HttpRequest bez jména presenteru, vyhodí to Notice.

David Grudl
Nette Core | 8228
+
0
-

Naja, mam tam buginu