Overenie Username a Emailu či už existujú v Db ANEB Pokus o registráciu!

A funguje ti to? To je podstatná otázka. Nicméně mám dvě výtky:

1. postovací hodnoty bych určitě nebral způsobem $_POST[‚uname‘] z důvodu validace escapování.
2. to ověřování emailu a jména v modelu se mi nezdá. Mělo by to být ve smyslu, že pokud existuje záznam, pak vrátím záznam, jinak např. do proměnné uložím že neexistuje. I to bych ale přesunul do presenteru, protože zde by měla probíhat aplikační logika a model by měl sloužit jen jako nástroj pro výběr dat v tomto případě.

Metoda serSuccessRegisterSubmitted je totálně špatně. Doporučuji si přečíst dokumentaci o formulářích

Jaaa to asi vzdám s Nette absolútne tomu nedokážem prísť na rozum :) už sa stým hrám skoro mesiac a ešte som nedokázal vytvoriť ani len obyčajnú registráciu užívateľa dokonca ani login a dokonca ani neviem kam čo mám umiestniť čo kam patrí apod. viem je to na smiech ale fakt sa snažím no aj tu čo mi pred chvíľou odp. matej21

$userRepository = $this->userRepository;
$form->addText('email', 'E-mail')->addRule(function($input) use($userRepository) {
    return !$userRepository->usermailexists($input->value);
}, 'Tento email jiz nekdo pouziva!');

Kam to mám umiestniť? neviem tak som to pacol sem:

$form->addText('umail', 'Emailová adresa:') // umail as email
		     ->addRule(function($input) use($userRepository )
		     {
    			return !$userRepository->usermailexists($input->value);
			 }, 'Tento email jiz nekdo pouziva!')
		     ->setRequired('Email je povinný!')
		     ->addRule(Form::EMAIL, 'Zadajte prosím, platnú emailovú adresu!'); input

Alebo aj toto:

6. pokud uz budes neco overovat az v onSuccess metode, tak tu chybu nezobrazuj pres flashMessage, ale pridej ji do formu $form->addError('')

Tak som to dal takto

public function UserSuccessRegisterSubmitted(Form $form)
	{
		try{

			$values = $form->getValues();
			$this->users->insert($values);

		} catch(Nette\Security\AuthenticationException $e) {
			 $form->addError($e->getMessage());
			return;
		}

	}

Nič žiadne, že email už existuje proste zapíše do DB a hotovo!

Dokumentáciu,fórum a Planette stále dookola čítam a robím podľa nej všetko je ok ale akonáhle niečo sám som ako na opustenom ostrove… :)
Ako tu toto fórum číslo 1 nápomocné a ochotné no nebaví ma sa ustavične pýtať hlúpe veci či to mám dobre ako to a ako hento…
škoda, že nie je viac video tutoriálov napr. Registrácia/Prihlásenie/Odhlásenie atď…
Možno sa to časom naplní ale fakt už neviem či som vekom osprostel :) alebo ja neviem, že nedokážem zachytiť pointu tohto skvelého a asi aj najlepšieho Frameworku…

P.S: Nerobí niekto náhodou za pár € doučovanie v na SK okolie Nitra?
rád by som sa zaúčal :) FAKT!

Nebylo by lepší mít na příslušném sloupci v DB unikátní klíč? Tak se ti o kontrolu duplicity postará databáze a ty jen ve zpracování zachytíš výjimku s příslušným kódem:

public function UserSuccessRegisterSubmitted(Form $form)
{
	try {
		$values = $form->getValues();
		$this->users->insert($values);
		$this->flashMessage(...);
		$this->redirect(...);
	} catch (\PDOException $e) {
		if ($e->getCode() === 23000) {
			// pokud je chyba v duplicitě klíče, přidáme chybu do formu
			$form->addError(...);
		} else {
			// pokud jde o jinou chybu, necháme ji probublat výš
			throw $e;
		}
	}
}

Pak se obejdeš bez vlastního validačního pravidla.

Ďakujem Vám všetkým za odpovede a za pomoc…
Poskladal som to takto dokopy a funguje zatiaľ ako má!
Prosím Vás mohli by ste to prekontrolovať? Vopred Ďakujem…
Môj UserRepository:
Chyba bola v tom, že za ->where(‚umail‘) som mal ešte dosadiť ,$umail asi takto

return $this->findAll()->select('umail')->where('umail', $umail)->fetch();

Keď som to tam dosadil začalo to vyhadzovať tie hlášky a nekonal sa žiadny zápis až som zmenil mail a potom sa pekne zapísalo do DB a zároveň vykonalo aj redirect

<?php
/**
 * Tabulka user
 */
class UserRepository extends Nette\Object
{
	/** @var Nette\Database\Connection */
	private $database;

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

	public function findAll()
	{
		return $this->database->table('users');
	}

	public function findById($uid)
	{
		return $this->findAll()->get($uid);
	}

	public function insert($values)
	{
		return $this->findAll()->insert($values);
	}

	public function userexists($uname)
	{
		return $this->findAll()->select('uname')->where('uname', $uname)->fetch();
	}
	public function usermailexists($umail)
	{
		return $this->findAll()->select('umail')->where('umail', $umail)->fetch();
	}

}

Ešte priložím môj UserPresenter

<?php
namespace FrontModule;

use Nette\Application\Ui\Form,
    Nette\Utils\Html,
    Country,
    DatePicker,
    UserRepository;

class UserPresenter extends \BasePresenter
{
	/** @var UserRepository */
	private $users;

	public function inject(UserRepository $users)
	{
		$this->users = $users;
	}

	protected function createComponentUserRegisterForm()
	{
		$userRepository  = $this->users;

		$form = new Form;

		$form->addText('uname', 'Užívateľské meno:')
		     ->setRequired('Prezývka je povinná!')
		     ->addRule(function($input) use($userRepository)
		     {
		     	return !$userRepository->userexists($input->value);
		     }, 'Toto užívateľské meno je obsadené prosím vyberte si iné!');

		$form->addText('umail', 'Emailová adresa:')
		     ->setRequired('Email je povinný!')
		     ->addRule(Form::EMAIL, 'Zadajte prosím, platnú emailovú adresu!')
		     ->addRule(function($input) use($userRepository)
		     {
    			return !$userRepository->usermailexists($input->value);
			 }, 'Táto emailová adresa je už v systéme zaregistrovaná!');

		$form->addPassword('upass', 'Heslo:')
		     ->setRequired('Heslo je povinné!')
		     ->addRule(Form::MIN_LENGTH, 'Heslo musí obsahovať min. %d znaky', 4);

		$sex = array('m' => 'Muž','f' => 'Žena',);
		$form->addRadioList('ugender', 'Pohlavie:', $sex)
		     ->setRequired()
		     ->getSeparatorPrototype()->setName(NULL);

		$form->addDatePicker('birthDate', 'Dátum Narodenia:')
    		 ->setAttribute('class', 'birthDate')
    		 ->addRule(Form::VALID, 'Entered birth date is not valid!');

		$ucountries = $this->CountryList();
		$prompt = Html::el('option')->setText('Vyberte Krajinu')->class('prompt');
		$form->addSelect('ucountry', 'Krajina:', $ucountries)
		     ->setPrompt($prompt)
		     ->setRequired();

		$form->addCheckBox('uagree', Html::el('span')
			 ->setHtml('Súhlasím s <a href="../docs/Terms" target="_blank">Podmienkamy</a>'))
		     ->addRule(Form::EQUAL, 'Musíte súhlasiť s podmienkamy!', TRUE);

		$form->addSubmit('uregme', 'Registrovať');
		 $form->onSuccess[] = $this->UserSuccessRegisterSubmitted;

		$form->addProtection();

		return $form;
	}

	public function UserSuccessRegisterSubmitted(Form $form)
	{
		//$values = $form->getValues();
		//$this->users->insert($values);
		try {
				$values = $form->getValues();
				$this->users->insert($values);
				$this->flashMessage(....);
				$this->redirect('Index:index');
		} catch (Nette\Security\AuthenticationException $e) {
			if ($e->getCode() === 23000) {
				// pokud je chyba v duplicitě klíče, přidáme chybu do formu
				$form->addError(...);
			} else {
				// pokud jde o jinou chybu, necháme ji probublat výš
			throw $e;
			}
		}
	}
}

Tak to s Nette teda skúsim odznova Možno, že o pár mesiacov ba i týždňov sa budem tomu smiať čo som sa tu pýtal a aké chyby som robil … No nejak sa to naučiť človek musí! Ďakujem Chalani…

Ešte som sa chcel spýtať ako riešite pri registrácii zabezpečenie a zahashovanie hesla? v presentery? niečo ako calculateHash ? v Sandboxe?

k ukladani hesel: http://www.michalspacek.cz/…profit-devel

ke kazdymu heslu pouzij jinej salt, pouzi alespon sha256 (nejlepe vsak bcrypt (resp. crypt)), jak je na tech slajdech

kde si zjistil ze to nic nehashuje? mam ten kod brat doslovne, ze jsi to zjistil dle dump($values);? v tom bude samozrejme heslo v plaintextu, ten hash je navratova hodnota te fce passwordHasher

jo a kdyz tu funkci passwordHasher pouzivas nestaticky, tak u ni nemej to slovo static :)

Editoval matej21 (27. 3. 2013 0:26)

no ale ty tam nic nedelas s tim vysledkem s passwordHasher.. treba tohle:

$values->password = $this->salt->passwordHasher($values->password);

Až na to že se to nerovná, ale znamená to, že poli $values do klíče password (viz $values->password) přiřadíš zahashované heslo.