Začiatočnícke otázky (Tovarnička vs. Presenter)

Zdravím,
obecně se doporučuje single-responsibility princip. Proto by bylo nejlepší kdyby jsi rozdělil SignPresenter na více presenterů: AuthenticationPresenter (login, logout), ResetPasswordPresenter, RegistrationPresenter (registrace, aktivace) Toto má svá omezení.

Pokud potřebuješ vykreslit přihlašovací komponentu ve všech presenterech, pak AuthenticationPresenter ztrácí význam. Uděláš si AuthenticationLoginControl a logiku (chápej jako logiku, která komunikuje s modelem!) nacpeš tam. V BasePresenteru vytvoříš pomocí createComponentAuthenticationControl()… komponentu a vykreslíš v layoutu. Podobně AuthenticationLogoutControl atd.

Ber to jako nástin možností. Není totiž možné říct, abys to dělal tak nebo jinak. Záleží na potřebách aplikace.

@echo & @matej21 Vďaka za reakcie :)
Trocha upresním prečo som sa na to pítal.
Všetky spomenuté formuláre chcem použiť iba na jednom mieste nikde inde ich nechcem umiestnovať 2×.
Preto som sa pýtal, či je lepšie továrnička s formulárom alebo formulár v presenter-u,
keďže som chcel mať fajn url v tvare http://www.domena.xx/sign/in atď… tým, že ich rozstrúsim budem mať aj inú tvar url nie? čiže už to bude http://www.domena.xx/ authentication/in atď…

Editoval SontoEremo (22. 6. 2014 19:19)

@matej21 Takže si vytvorím dajme tomu Továrničku s napr. UserRegister.php a vňom

use Nette\Application\UI\Form;

class UserRegister extends Nette\Object
{
    private $database;

    public function __construct(Nette\Database\Connection $database)
    {
        $this->database = $database;
    }

    public function createComponentSignUpForm()
    {
        $form = new Form;
        ....
        return $form;
    }

    public function signUpFormSuccessSubmited($form)
    {
        ...
    }
}

atď … a to potom predať SignPresenter-u a nakoniec vykresliť v Šablone napr. in.latte ?

@echo:

1. do formu nemusis predavat this a name.
2. nepouzivej zadny ->isSuccess()… ale onSuccess event
3. tohle myslim @SontoEremo uz umi a ma. tahle diskuze je o refaktoringu toho kodu pryc z presenteru

Editoval matej21 (22. 6. 2014 21:36)

@echo & @matej21
Tak po poriadku chalani :) pretože som sa dosť pomýlil takže normálne si vytvorím
AuthenticationPresenter kde budem riešiť logout + login a zas napr. RegistrationPresenter pre registráciu atď…
Šablonu pak řešit přímočaře:

{control loginForm}

Ale kde potom budem definovať Css+Html štruktúru? ako mám teraz napr. login

<div id="signup-page-content">

		{form signInForm class=>"form"}
			<h1 class="block-heading">Prihlásenie</h1>
			<div n:foreach="$form->errors as $error" id="errors" class="alert alert-danger">
			    <strong >{$error}</strong>
			</div>
			<div n:foreach="$flashes as $flash" id="flashes" class="alert alert-danger flash {$flash->type}">
			    <strong >{$flash->message}</strong>
			</div>
			<div class="input-icon mbl">
				<i class="fa fa-user"></i>
				{input username}
			</div>
			<div class="input-icon mbl">
				<i class="fa fa-key"></i>
				{input password}
			</div>
			{input signIn class=>"btn btn-lg btn-red  btn-block"}
		{/form}
		<div class="sign-footer">
			<ul class="list-unstyled">
				<li><a n:href="Sign:reset">Zabudli ste heslo?</a></li>
				<li>Ešte nemáte účet? <a n:href="Sign:up">Vytvorte si ho</a></li>
				<li>Nedostali ste aktivačný email? <a n:href="Sign:resend">Pošlite si ho znova</a></li>
			</ul>
		</div>
	</div>

je o refaktoringu toho kodu pryc z presenteru Tak v presenteru alebo preč z presenteru :)

Editoval SontoEremo (22. 6. 2014 21:51)

@matej21 tebe veľmi moc Ďakujem za rady.
Ale mal by som prosbu mohol by si mi ukázať ako to s tou továrničkou? nejaký rýchli kód ako to všetko vytvoriť pretože som prešiel všetko ale nejak mi to stále neni jasné :)

@SontoEremo zkus si to :) kdyz ti budu posilat hotove reseni, tak je to k nicemu. Hezky priklad je v tomto clanku, na ktery uz jsem odkazoval

Proste si vytvoris tridu RegisterationFormFactory, do create metody das inicializaci formulare, vytvoris si tam nejakou processForm metodu, kde se form zpracuje a ulozi. Tuhle factory registrujes jako sluzbu, injectnes do presenteru a v prislusne createComponent* metode zavolas $his->registrationFormFactory->create() … ale to vsechno je v tom clanku, tak si ho precti :)

a jeste jeden prispevek pro inpiraci: https://forum.nette.org/…nebo-tovarna

Editoval matej21 (22. 6. 2014 22:07)

@SontoEremo: skoro dobre :) jen nepouzivej mimo presentery injectovani pomoci anotace inject. to se hodi (jen) do presenteru (je tady uz spoustu diskuzi ohledne toho a nebudu zacinat dalsi proc tomu tak je). takze si ten repozitar vyzadej v konstruktoru, stejne jako to delas s tim databazovym spojenim. Jinak tam na prvni pohled nevidim chybu.

Neinjectujes soubory, ale objekty (instance trid)
Vic konstruktoru mit nemuzes, proste do jednoho konstruktoru pridej vice parametru

protected $userRepository;

protected $userManager;

public function __construct(\App\Model\UserRepository $userRepository, \App\Model\UserManager $userManager)
{
	$this->userRepository = $userRepository;
	$this->userManager = $userManager;
}

Predsa len ešte jedná otázka
v presenteri som uvádzal $template->verifyLink = $this->presenter->link(‚//Sign:verify‘, array(‚token‘ ⇒ $token,));
Teraz by som asi mal namiesto $this->presenter->link() mal používať niečo iné ale čo?

@Šaman
Vyhadzuje mi vždy
Call to undefined method App\components\RegisterFormFactory::plink()

@Šaman ja mám RegisterForm a vňom

<?php
namespace App\components;

use Nette\Object;
use Nette\Application\UI\Form;
use Nette\Utils\Html;
use Nette\Security as NS;
use Nette\Mail\Message;
use Nette\Mail\SmtpMailer;
use Nette\Utils;
use Nette\Templating\FileTemplate;
use Nette\Latte\Engine;
use Nette\Templating\Helpers;

class RegisterFormFactory extends Object
{
	/** @persistent */
    public $backlink = '';

    protected $userRepository;

	protected $userManager;

	public function __construct(\App\Model\UserRepository $userRepository, \App\Model\UserManager $userManager)
	{
	    $this->userRepository = $userRepository;
	    $this->userManager = $userManager;
	}

    public function createSignUpForm()
    {
    	$repository = $this->userRepository;
		$sign_Up = new Form();
		$sign_Up->addProtection();
		$sign_Up->addText('username')
					 ->setAttribute('placeholder', 'Užívateľské meno')
					 ->setAttribute('class', 'form-control')
					 ->addRule(Form::MIN_LENGTH, 'Užívateľské meno musí obsahovať aspoň %d znaky', 4)
					 ->addRule(Form::MAX_LENGTH, 'Užívateľské meno môže obsahovať max. %d znakov', 32)
					 ->addRule(function($input) use($repository)
		             {
		                return !$repository->userexists($input->value);
		             }, 'Toto užívateľské meno je obsadené prosím vyberte si iné!');
		$sign_Up->addText('email')
					->setAttribute('placeholder', 'Emailová adresa')
					->setAttribute('class', 'form-control')
					->addRule(Form::EMAIL, 'Zadajte prosím, platnú emailovú adresu!')
					->addRule(function($input) use($repository)
		             {
		                return !$repository->usermailexists($input->value);
		             }, 'Táto emailová adresa je už v systéme zaregistrovaná!');
		$sign_Up->addPassword('password')
					->setAttribute('placeholder', 'Heslo')
					->setAttribute('class', 'form-control')
					->addRule(Form::MIN_LENGTH, 'Heslo musí obsahovať aspoň %d znaky', 4)
					->addRule(Form::MAX_LENGTH, 'Heslo môže obsahovať max. %d znakov', 32);
		$sign_Up->addPassword('confirm_password')
					->setAttribute('placeholder', 'Potvrďe Heslo')
					->setAttribute('class', 'form-control')
					->addRule(Form::EQUAL, 'Zadané heslá nie sú zhodné!', $sign_Up['password']);
		$sign_Up->addSubmit('signUp', 'Vytvoriť účet');
        $sign_Up->onSuccess[] = $this->signupsuccessForm;
        return $sign_Up;
    }

    public function signupsuccessForm(Form $sign_Up) {
    	$values = $sign_Up->getValues();
        if ($values->password === $values->confirm_password){
            $user = $this->userManager;
            $token = Utils\Random::generate(32);
            $userip = $_SERVER['REMOTE_ADDR'];
            $user->add($values->username, $values->email, $values->password, $token, $userip);
			$template = new FileTemplate(__DIR__ . '/../templates/Activate/email.latte');
			$template->registerFilter(new Engine);
            $template->registerHelperLoader('\Nette\Templating\Helpers::loader');
            $template->user = $user;
            $template->verifyLink = $this->presenter->plink('//Sign:verify', array(
                'token' => $token,
            ));

            $mail = new Message;
            $mail->setFrom('VistmoLabs <vistmolabs@vistmolabs.org>')
                 ->addTo($values['email'])
                 ->setSubject('Aktivácia účtu')
                 ->setHtmlBody($template);

            $mailer = new SmtpMailer(array(
		        'host' => 'smtp.vistmolabs.org',
		        'port' => 465,
		        'username' => 'vistmolabs@vistmolabs.org',
		        'password' => 'WS83ZYF6427g5G9',
		        'secure' => 'ssl',
			));
        	$mailer->send($mail);
            $this->flashMessage(Html::el('div')
             	 ->setHtml('Registrácia prebehla úspešne. Na zadanú Emailovú adresu <strong class="email-strong"><br>' . $values->email . '<br></strong> Vám bol zaslaný aktivačný link, po úspešnej aktivácii sa budete môcť prihlásiť.'));
            $this->redirect('finish');
        } else {
            $signup_form->addError($this->getMessage());
        }
	}
}

problém mi robí táto časť

$template->verifyLink = $this->presenter->plink('//Sign:verify', array(
               'token' => $token,
           ));

@Šaman dík spravil som to podľa tvojho formulára a všetko funguje ok ale chcem sa spýtať v šablone Components/LoginForm/loginForm.latte chcem uviesť link ale vyhadzuje to chybu

<ul class="list-unstyled">
				<li><a n:href="#">Zabudli ste heslo?</a></li>
				<li>Ešte nemáte účet? <a n:href="#">Vytvorte si ho</a></li>
				<li>Nedostali ste aktivačný email? <a n:href="#">Pošlite si ho znova</a></li>
			</ul>

predtým v presenter-y stačili <li><a n:href=„Sign:reset“>Zabudli ste heslo?</a></li> ale tu neviem ako na to.

A ešte sa chcem spítať naozaj je potrebné mať komponentu keď prihlasovací a vlastne všetky formuláre budú len na jednom mieste a nikde inde?