Banalita – default action

default je kouzelná v tom, že se nikam psát nemusí. Objeví se leda v názvu souboru a nějaké metody, ale odkazování je jenom Modul:Presenter: a ne Modul:Presenter:default.

Když je to konstanta, nestačí ji změnit v BasePresenteru?

Edit: je tam self, takže nešlo. Škoda.

Editoval Mikulas Dite (22. 5. 2011 23:00)

To jo, ale takto to nejde udělat ani v BasePresenteru, takže mi Nette určuje že to bude vždycky default, a nejde to změnit (pokud nechci zkopírovat tyhle dvě metody do BasePresenteru) – a to je přesně to co Nette dělat nechce ne? :)

@Mikulas: a objeví se taky v url, což je to co mi vadí…

Editoval Aearsis (22. 5. 2011 23:22)

Proč bys to chtěl předefinovat globálně? Proč ti jiný název, než default, přijde lepší?

22: S routami to nesouvisí, to by tam ta konstanta nebyla potřeba vůbec :)

Bez urážky: /page/show/contact se ti líbí? A co bude na adrese /page/show, resp. /page? jsi si jistý, že zrovna defaultAction je to, co potřebuješ změnit?

taková URL standardně nevznikne, protže defaultní akce se v URL vynechá, takže by ti to mělo hodit /presenter/id – pokud máš id

@22: To ne, potom by nebylo možné rozeznat ID od jména akce.

@Aearsis: Defaultní akce je od toho, že se nemusí uvádět. Pokud plánuješ nějakou akci zobrazovat v 99% adres, pravděpodobně by to neměla být ta defaultní.

Z toho důvodu se s adresou /presenter/default/id prakticky nesetkáš – nedávalo by to smysl.

Editoval _Martin_ (23. 5. 2011 0:16)

@Aearsis: Potom je to otázka nastavení routy, vzhledem k tomu, že jde o akci s parametrem. Tedy něco takového:

$router[] = new Route('<presenter>/<action>/<id=0>', 'Page:show');

Zkus ještě zvážit ten tvar adres /page/show/page-name – /page/show neodráží logickou strukturu, ale spíš technickou implementaci webu.

@Aearsis: Můžeš nějak víc popsat tu appku? Jako presentery, akce a jejich parametry. Možná by to chtělo jinou strukturu aplikace a nebo vlastní routy – nedokážu soudit z jednoho příkladu, ale „potřeba změny defaultAction + její použití s parametrem + ve všech presenterech“ na mě působí nezvykle.

Přijde mi to jako legitimní požadavek, ač autoro se tím snaží vyřešit něco jiného. Možná náhrada default může být třeba slovo index.

(Však si vzpomeňte, jak to bylo dříve, default.html ale index.html vyhrál :D :D)

Editoval hrach (23. 5. 2011 9:24)

To, co řešíš, je nesmysl, máš špatně napsané routy a s konstantou v presenteru to nijak nesouvisí.

V presenteru Page měj akci renderDefault(), která ti vypíše seznam položek a renderShow($id), která ti vypíše jednu konkrétní položku.

Tyhle dvě routy ti pak obstarají ideální URL:

$router[] = new Route('page', array(
	'presenter' => 'Page',
	'action' => 'default',
));

$router[] = new Route('page/<id>', array(
	'presenter' => 'Page',
	'action' => 'show',
));

Pokud bys to chtěl zobecnit s tím, že máš víc presenterů, které mají stejné default a show akce + nějaké další jiné, tak takto:

$router[] = new Route('<presenter>/<action redakce|archiv|kompletni|xml|kratke-zpravy|komentare>', array(
	'action' => 'default',
));

$router[] = new Route('<presenter>/<id>', array(
	'action' => 'show',
));

Budto jsem totalne mimo nebo fakt nevim ale vubec nechapu wo co go!

Můžeš pls uvés příklad tak 5ti adres na jake akce(+presentery) a parametry je chces?

Ok, zkusím to polopaticky :)

Chci, aby:

:Page: == :Page:show,null
:Page:show,5 == :Page:show,5

:Gallery: == :Gallery:show,null
:Gallery:show,slozka/ == :Gallery:show,slozka/

Prostě bych rád, abych nemusel definovat akci default, která vykoná fci stejnou jako show, akorát bez parametru. Buď musím načtení a zobrazení stránky přesunout do jiné funkce a volat ji z obou, nebo přes changeAction, to ale změní url v prohlížeči, nebo zajistit aby se nikde neodkazovalo na :Page: a dopsat všude :Page:show. Tohle všechno za mě udělá změna dvou self na static za cenu nulového BC breaku, nepatrného zpomalení (static je u mě cca o 5 ns pomalejší), nette už nebude přikazovat že default akce je vždycky default, a určitě by se ještě nějaké pro našlo.

Editoval Aearsis (23. 5. 2011 18:33)

if ($id == null) {
        $this->template->page = \Model\Page::find(array("id" => \Model\Setting::find("Homepage id")));
        if ($this->template->page == null)
           throw new \Nette\Application\BadRequestException("Domovská stránka nenalezena! Špatné nastavení!", 404);
} else {
        $this->template->page = \Model\Page::find(array("id" => $id));
        if ($this->template->page == null)
        	throw new \Nette\Application\BadRequestException("Stránka nenalezena", 404);
}

... kontrola oprávnění, breadcrumbs, titulek atp. ...

Editoval Aearsis (23. 5. 2011 18:36)

Ale spolu s tim budu muset znovu zkopírovat ten konec fce, kontrolu, nastavení titulku, breadcrumbs, zalogování zobrazení atd..

Některé části kódu můžeš dát do společných metod, viz. životní cyklus presenteru.

O beforeRender jsem vědel, o afterRender ne. Děkuji, tím odpadají všechny námitky (kromě té že mi to nette přikazuje, ale s tím se smířím) :)