Redirect v BasePresenteru ve startup() → Příliž mnoho přesměrování

Tvůj „globální“ účinek je až příliš globální, neboť se vztahuje i na akci Sign:in. Tu je třeba nějak vyčlenit. Buď explicitně podmínkou nebo např. tak, že SignPresenter bude dědit z jiného BasePresenteru, ve kterém tato kontrola nebude.

Díky za rychlou odpověď. Je to na mě moc obecně napsané.
K prvnímu způsobu: Mohl bys mi prosím nastínit, kam a jakou podmínku napsat aby se pro Sign:in neprováděla kontrola ve startup() BasePresenteru (jestli jsem to pochopil dobře?)
K druhému způsobu: Aha :) Tak si to zatím zkusím napsat pomocí dalšího BasePresenteru

To je asi lepší způsob, osobně používám:

[BasePresenter] → (presentery nevyžadující přihlášení)
[BasePresenter] → [SecuredPresenter] → (presentery vyžadující přihlášení)

test na autentikaci (no a i základní autorizaci) je pak prostě v SecuredPresenter::startUp(). V BasePresenteru je teda to, co je společné pro veřejné i neveřejné části.

hAssassin, Nox: To je pravda – to mě nenapadlo. Ostatně mě toho ještě dlouho moc napadat nebude, protože cca před měsícem jsem začal s OOP a to myšlení si ještě musím zažít.

Díky všem za rychlé reakce – je to paráda.

@thm > sorry ale podminka je podle me cisty zlo, protoze:

1. v podmince se ti vyskytuje primo nazev presenteru i akce, pokud se neco z toho rozhodnes menit musis upravovat nekolik souboru (prejmenovani, prepis presenteru, oprava podminky), kdezto v druhem pripade ti staci jen v prezenteru zmenit nazev tridy ze ktery dedi a je to.
2. pokud mas jen jediny verejny presenter, tak OK, ale pokud jich mas treba vic (typicky Front je vetsina verejna, pouze par presenteru je neverejnych). To tam budes vypisovat skutecne vsechny presentery a akce? :-)

@llsm >

A tam mi nejake zbesile neprehledne dedeni presenteru prijde horsi nez udelat jednu osklivou podminku

Neprehledne dedeni? Vzdyt ti pribyde jen jediny presenter? co je na tom neprehlednyho? Naopak ta podminka je dost neprehledna protoze je zahrabana nekde v nejaky metode v nejakym souboru a to se pak tezko hleda. Navic je sama neprehledna, takze kdyz ji nekdo uvidi, rekne si „whaaat?“.

Navic u slozitejsich struktur se podobne veci resi vetsinou pomoci ACL a to je pak cele jina pisnicka.

Bacha, neplest si autentizaci (tedy overeni, zda je uzivatel prihlaseny) s autorizaci (coz dela ACL ;).

Editoval hAssassin (10. 4. 2013 16:07)

@Aurielle > neni tohle trochu pekna prasarna? Jako fungovat to bude, ale neporusuje to vsechno co muze na dedicnosti? a jaky je pak logicky rozdil mezi tim mit dva base presentery? Tady jsou prece taky, jen jsou zkryty (resp jeden z nich je puvodni z nette).

Navic pres ty dva presentery (Base a BaseSecured) to ma i dalsi vyhodu v tom, ze pro prihlaseny uzivatele obvykle potrebujes i neco vic nez jen to overeni/presmerovani. Napr. ten handleLogout() jak sem zminoval vyse je presny kandidat, ktery ma v BaseSecured svoje misto (naopak v Base nema co delat). Nebo nejaka komponenta ktera se zobrazuje pouze prihlasenymu uzivateli, nejaky specialni polozky menu, … A tady ta vyhoda uz je zrejma…

Ještě existuje jedna možnost a sice v BasePresenteru vyčlenit kontrolu a případnou redirekci do zvláštní metody (volané z BasePresenter::startup) a tuto metodu pak ve všech Presenterech, kde nechci tuto kontrolu, nebo ji chci nějak omezit, překrýt. Tj.

class BasePresenter extends Nette\Application\UI\Presenter
{
	public function startup()
	{
		parent::startup();
		$this->ensureAccessible();
	}

	protected function ensureAccessible()
	{
		if (!$this->getUser()->isLoggedIn()) $this->redirect('Sign:in');
	}
}

… a v SignPresenteru mít:

	protected function ensureAccessible()
	{
		if ($this->action !== 'in') parent::ensureAccessible();
	}

llsm wrote:

Musis kontrolovat tedy i jestli jsi na prihlasovaci strance a pokud ano, tak aby se redirect neprovedl. Neco jako:

<?php
 if ((!$this->user->isLoggedIn()) && ($this->reflection->name != 'SignPresenter' && $this->getAction() != 'in')) $this->redirect('Sign:in');
?>

Jen doplním, že místo $this->reflection->name je výrazně lepší zavolat $this->getName() což vrací název aktuálního presenteru.