Ako overit v ktorom presenteri sa nachadzam

buksy

Chcem presmerovat neprihlasenych uzivatelov, preto som do zakladnej triedy od ktorej dedia vsetky presentery pridal presmerovanie, problem je ze z nej dedi aj „Sign“ presenter a ten sa potom snazi presmerovat sam na seba a vytvori nekonecny cyklus, ako tomu zabranit?

<?php
namespace WebModule;

abstract class WebBasePresenter extends \BasePresenter
{
	protected function startup()
	{
	    parent::startup();

	    // Potreboval by som nieco ako !$this->getUser()->isLoggedIn() && $this->getName() != "Sign"
	    if (!$this->getUser()->isLoggedIn()) {
	        $this->redirect('Sign:in');
	    }
	}
}
?>

Michal Vyšinský

// Potreboval by som nieco ako !$this->getUser()->isLoggedIn() && $this->getName() != „Sign“

No a proč to tam nedáš?

Editoval CherryBoss (9. 9. 2013 15:09)

David Matějka

doporucuju spis upravit strukturu presenteru, jak od sebe budou dedit – treba:

BasePresenter
	SignPresenter
	SecuredPresenter - az tady budes overovat prihlaseni
		FooPresenter
		BarPresenter
		....

buksy

@CherryBoss: Lebo to som si len tak tipol ale nevidel som to v dokumentacii (nevedel som ze to existuje), potom som nasiel metodu $this->getRequest()->getPresenterName() :)

@matej21: Dobry napad, dakujem

enumag

@matej21: Zrovna na přihlášení tohle nepotřebuješ, Nette na to má anotaci @User(loggedIn). Osobně žádný SecuredPresenter už dávno nepoužívám.

duke

@enumag: Předpokládám, že případné přesměrování na Sign presenter či jinam pak řešíš přes Application::onError. Je to tak?

enumag

@duke: Ne, to dělám v ErrorPresenteru na základě typu vyhozené exception.

Editoval enumag (9. 9. 2013 19:44)

n.u.r.v.

Ahoj, a jak moc špatná je moje metoda, když mám SignPresenter, HomepagePresenter a další a v každém presenteru (kde je přihlášení vyžadováno) mám v metodě startup() toto:

public function startup() {
        parent::startup();

        if (!$this->getUser()->isLoggedIn()) {//pokud není uživatel přihlášen
            $this->redirect('Sign:in');
        }
....
....
}

A naopak v SignPresenteru mám:

public function startup() {
        parent::startup();
        if ($this->getUser()->isLoggedIn()) {//pokud je uživatel přihlášen
            $this->redirect('Homepage:default');
        }
}

díky…

enumag

@n.u.r.v.: Jako špatnou bych ji nenazval (taky jsem to dříve používal), jen to dle mého soukromého názoru není best practice.