Screencast – kontaktný formulár: ‚contactForm‘ does not exist

chybí ti tady n

protected function createComponentCotactForm($name

use Nette\Application\UI\Form, Nette\Forms\Form; – tohle nemůže fungovat… → to druhé umaž

Ještě jedna věc – tvoříš li komponentu přes return, pak jsou argumenty $this a name zbytečné: uprav na toto:
function createComponentCo*n*tactForm($name){return new ContactForm($this, $name);}
 function createComponentCo*n*tactForm(){return new ContactForm;}

Editoval bojovyletoun (8. 6. 2011 19:51)

Ahoj,

@vvoody nie ze by sa mi nechcelo to opravit :-) (ano je tu problem so zmenami verzii, API atd…), no nestiham vsetky zmeny sledovat (ale 20teho statnice a potom hura hokej :-)) cize neviem co sa menilo a co nie a ako to urobit po novom. Samozrejme by som bol rad keby ludia ktori si zmeny vsimaju to bud opravili aspon v dokumentacii (zas taky problem to nie je) popr. sa mozu ozvat na forum ku kazdemu scrcastu je vlakno a predpokladam ze autor konkretneho scrcastu by to do dokumentacie zapracoval. S tym, ze by aktualne kody boli napr na githube.

Chvíľu som zápasil (úspešne) s neon configom. Zapísal som si do neho mail a subject tak ako tomu je v screencaste kde si to zapísal do ini configu. Mňa by zaujímalo ako by sa malo pristupovať k údajom v neon configu. Mne sa to podarilo nasledovne:
config

common:
	php: # PHP configuration
		date.timezone: Europe/Prague
		# session.save_path: "%tempDir%/sessions"
		# zlib.output_compression: yes
		# default_charset: "UTF-8"

	services:
		authenticator:
			class: UsersModel

		robotLoader:
			run: true
	mail:
		to: "moj@mail.sk"
		subject: "Kontaktný formulár"

kúsok prezenteru:

<?php
    protected function sendContactEmail($values)
    {
        $mail = new Nette\Mail\Message;

        //$mailCOnfig = Nette\Environment::getConfig('mail');
	$mailConfig = Nette\Config\NeonAdapter::load(APP_DIR . '\config.neon');

        $mail->setFrom($values['email']);
        $mail->addTo($mailConfig['common']['mail']['to']);
        $mail->setSubject($mailConfig['common']['mail']['subject']);

        $mail->setBody($values['message']);

        $mail->send();

        $this->flashMessage('Správa bola odoslaná');

        $this->redirect('this');
    }
?>

Je to správny spôsob alebo existuje nejaký slušnejší?

_Martin_ napsal(a):

Bylo by lepší, pokud bys závislosti dodal z venku – tj. ono Dependency Injection.

Z vonku triedy alebo stačí z vonku funkcie? Nie som si celkom istý či ono DI znamená injectovanie len do tried (presnejšie asi do ich inštancií) alebo sa dá považovať za DI aj keď si len bokom (pod tou istou triedou) spravím o funkciu viac?

<?php

    public function contactFromOkClicked(\Nette\Forms\Controls\SubmitButton $button)
    {
        $values = $button->getForm()->getValues();
        $mailConfig = $this->getMailConfig();
        $this->sendContactEmail($values,$mailConfig);
    }

    protected function sendContactEmail($values,$mailConfig)
    {
        $mail = new Nette\Mail\Message;

        $mail->setFrom($values['email']);
        $mail->addTo($mailConfig['to']);
        $mail->setSubject($mailConfig['subject']);
        $mail->setBody($values['message']);

        $mail->send();

        $this->flashMessage('Správa bola odoslaná');

        $this->redirect('this');
    }

    private function getMailConfig()
    {
	$config = Nette\Config\NeonAdapter::load(APP_DIR . '\config.neon');
	return $config['common']['mail'];
    }

}
?>

A musím sa ešte raz spýtať či nieje iný spôsob zistenia obsahu configu, lebo load(APP_DIR . ‚\config.neon‘); mi príde úplne divné kedže ten config už musel byť načítaný tak prečo ho načítavať znova, nieje už niekde uložený v nejakom poly?

Druhá otázka, patrí vôbec ten mail/subject do toho configu? Nedať ho radšej niekam inam?

Viem riešim skoro až nezmyselné drobnosti ale keď už zrevidovať ten screencast tak poriadne :) aby nováčikovia ako ja nezískali zlé návyky.