Předávání objektu akcím presenteru a metodě link

Počkat, tohle nejde? Chtěl jsem podobným způsobem předělat svoje routy, hlavně kvůli šablonám (<a n:href="Articles:show $article"> místo <a n:href="Articles:show $article->id">). Musim to vyzkoušet…

Já nevim jestli to funguje nebo ne, ale přišlo by mi to užitečný. Takhle ten objekt načítám dvakrát – jednou v routě, abych zjistil jestli existuje, a pak v presenteru (samozřejmě ho repository hledá jenom jednou, ale stejně mi to smrdí).

Při linkování ze strany Presenteru to padá tady, tedy v metodě argsToParams. Pokud objekt nemá metodu __toString() vracící například slug pro routu, tak to spadne na Recoverable erroru…

@Jan Voráček: tou metodou je právě zmiňovaný __toString() jak píše gmvasek tady

Taky se přidávám s tímhle požadavkem. Nechci si kurvit objekty, které strkám do rout (třeba entity) nějakou metodou __toString(), která je vůbec nemá zjaímat.

Bump, taky jsem pro.

Já si naopak říkám, proč by nemohl? Právě router rozhoduje o tom, jestli konkrétní cesta je ta správná a pokud kontroluješ existenci objektu až v presenteru, tak už to routeru nevrátíš.

Btw: Doctrine má tu výhodu, že když si v routeru potvrdím existenci nějakého objektu, tak si vrátím místo objektu jeho identifikátor. Díky IdentityMap a UnitOfWork se mi tak objekt načte jenom jednou, protože si řeknu přesně o něj.

$repo = $container->entityManager->getRepository('Node');
$router[] = new Route('/<node .*>', array(
	'nodeId' => array(
		'presenter' => 'Front',
		'action' => 'default',
		Route::FILTER_IN => function ($name) use ($repo) {
			$node = $repo->findByName($name);

			// když vrátí null, routa bude odmítnuta
			return $node ? $node->getId() : NULL;
		},
		Route::FILTER_OUT => function ($id) use ($repo) {
			return $repo->find($id)->getName();
		},
	)
));

A co s tím v presenteru?

public function actionDefault($nodeId)
{
	$node = $this->context->entityManager->getRepository('Node')->find($nodeId);
	// nevykonává se žádný SQL, protože IdentityMap už entitu s tímhle ID má načtenou
}

Takže co? Nepotřebuju předávat objekt, proč taky :)

Pro-Tip: Je taky možné si vytvořit nějaký Helper, na převádění vstupů na objekty (v BasePresenteru).

protected function entityFromParam($class, $paramName, $name = NULL)
{
	if (!isset($this->params[$paramName])) {
		return;
	}

	$repository = $this->context->entityManager->getRepository($class);

	$id = $this->params[$paramName];
	$entity = $repository->find($id);

	$this->params[lcFirst($name ?: $class)] = $entity;
}

Nyní můžu upravit svůj presenter takto:

protected function startup()
{
	$this->entityFromParam('Article', 'articleId');
	$this->entityFromParam('Poll', 'pollId');

	parent::startup();
}

public function actionDefault(Article $article)
{
	// nejsem si jistý tím type-hintem, ale nevím o ničem, co by tomu bránilo.
}

Jedna nevýhoda tu je, musí se tomu přizpůsobit generování odkazů

{link this, 10} <!-- nestačí -->
{link this, 'articleId' => 10}

Případ 1: Router vrátí NULL, Nette tedy použije nejvíce obecnou routu (nebo samo Nette hned vrátí BadRequestException, neexistuje-li daný presenter – pokud jsou routy nastaveny tak, že by třeba node odpovídal v obecné routě presenteru). V presenteru se při dotazu na konkrétní objekt vrátí také NULL, protože ID buď neexistuje nebo nemůže být string. A tam se už dá vyhodit BadRequestException.

HosipLan napsal(a):
Dovolím si tvrdit, že moc obecná řešení nemůžou být obecně dobrá :) (ve většině případů) Ale svůj účetl to plní, ne? Ušetřil jsem výkon a pracuji s objekty.

Pořád mám pocit, že je to zbytečně složité… Navíc třeba s Nette\Database něčeho takového nedocílíš, což mi přijde zvláštní, vzhledem k tomu že to je nativní knihovna v Nette…

wdolek napsal(a):

Spise jsem narazel na to, ze to neni vubec „lazy“ – pomerne dost aplikacni logiky resim uz v routeru, kam reseni podobnych veci proste nepatri (z filozofickeho hlediska).

Chápu o co ti jde, ale dalo by se o tom dlouze hádat, protože v routeru nic neřeším, ale předávám tam objekt, který se o to postará. Vždyť to jsou callbacky, v routeru se jenom zavolají :)

No a taky, proč by to mělo být lazy? Můžu to přece cachovat. Nette má moc krásně řešenou invalidaci.

spidy napsal(a):

Navíc třeba s Nette\Database něčeho takového nedocílíš, což mi přijde zvláštní, vzhledem k tomu že to je nativní knihovna v Nette…

Jakto, že nedocílím? Stačí si udělat primitivní IdentityMapu, třeba klidně v repozitáři a funguje mi to úplně stejně. Znáš DI, ne? :)

PS: Neobhajuju nemožnost pracovat s objekty, nebo naopak. Jenom nabízím řešení.

HosipLan napsal(a):

Chápu o co ti jde, ale dalo by se o tom dlouze hádat, protože v routeru nic neřeším, ale předávám tam objekt, který se o to postará. Vždyť to jsou callbacky, v routeru se jenom zavolají :)

Take chapu, jak to myslis :D problem je v tom, ze pravdepodobne ty callbacky nastavis nekde v bootstrapu, ne? Neumim si ted predstavit, jak zaridit, aby presenter sam rekl routeru, co mu ma predat – protoze v momente, kdy je ve hre samotny presenter je vlastne uz pozde ovlivnit chovani routeru (a dokonce je to i zbytecne).

Taktez reseni, ze by ten callback byla treba staticka funkce presenteru, kterou bys jen v callbacku nastavil, moc hezke reseni (byt funkcni) – jak si na tu statickou tridu napises unit test?

@**wdolek**: Tady přichází k řeči to škaredé složité DI :) Viz stará verze mého routeru, jako demonstrace.