Ako napísať registráciu v súlade s PDO?

Třeba kdyby/doctrine vyhodí Kdyby\Doctrine\DuplicateEntryException, tu si hned v nějaké např. přidávací metodě pro uživatele odchytávám. Zkontroluji, jestli se jedná o duplikaci na emailu nebo někde jinde (třeba nějaký username) a podle toho dál vyhodím svou App\DuplicateEmailException nebo App\DuplicateUsernameException apod. Jinak nechám vyhodit zpět původní exception z kdyby.

try {
	$dao->save($user);
} catch (\Kdyby\Doctrine\DuplicateEntryException $e) {
	if (checkSomehowIfExceptionIsBecauseOfDuplicateEmail($e)) {
		throw new \App\DuplicateEmailException($user, $e);
	} else {
		throw $e;
	}
}

No a ve formu si odchytávám už přímo tyhle moje výjimky.

Ono když je totiž jen kontrola podle toho, jestli už ten záznam v db není a má jít pak i editovat, tak je ta kontrola hned složitější. Při přeuložení bez změny by mi totiž form hodil chybu, že email už existuje.

Ale mňa zaujíma hlavne kód tej metódy register(). Ale díky za to video.

Editoval Čamo (8. 7. 2014 13:19)

@Šaman nene neinsertuji. Jen jsem to asi špatně napsal. Pokud by při editaci nechal jen tuhle kontrolu:

if($this->database->table(users)->where('name = ?', $values['name'])->fetch())
{
    $form->addError('Meno je už obsadené');
    $this->redirect('this');
}

tak formulář ukáže chybu, že jméno je obsazené, když ho nijak nezměním. Musel bych tu podmínku rozšířit a když těch „unikátních“ sloupců bude víc, tak se to zesložití a pro vkládání to bude jinak a pro editaci jinak. Proto mi přijde unique určitě lepší (spolu s tím, že do db to můžu vložit přímo atd…)

@Šaman

Při update samozřejmě unikátní index nevadí a nic nevyhazuje…

?? unikatni index se bere v potaz i pri update

To záleží jaké metody v repository používaš a jakou db používáš. Pokud Nette Database Context tak něco jako:

	$this->dbContext->table("user")->insert($values);

Editoval jenicek (8. 7. 2014 15:45)

David Kudera
Díky za odkaz, ale to nemám šancu rozkryť. Nechám to tak ako to je aj keď to bude zložitejšie pri väčšom počte unique indexov.

Editoval Čamo (8. 7. 2014 20:22)

Tak som to nakoniec zbastlil takto:

try
{
	$row = $this->database->table('users')->insert($arr);
	$this->flashMessage('Boli ste zaregistrovaní. Môžete sa prihlásiť');
	$this->redirect('Sign:');
}
catch(\Exception $e)
{
	$db = $this->database;
	if($db->table('users')->where('username', $values['username'])->fetch() && !isset($row) )
	{
		$this->flashMessage('Meno '.$values['username'].' je už obsadené. Vyberte si prosím iné.');
		$this->redirect('this');
	}
	elseif($db->table('users')->where('email', $values['email'])->fetch() && !isset($row) )
	{
		$this->flashMessage('Email '.$values['email'].' je už obsadený.');
		$this->redirect('this');
	}
	else
	{
		throw $e;
	}
}

Síce ak nastane nejaká iná výnimka(než duplicate entry) a zároveň už existuje daný uživateľ(čo by bola duplicate entry), hodí to chybne duplicate enty error a neošetrí tú skutočnú výnimku. Ale po opravení mena/mailu sa aj tá neošetrená vyhodí/zaloguje(ak teda ešte je čo logovať). Ale zase je to hádam dosť nepravdepodobné. Na viac zatiaľ nevydalo.

Editoval Čamo (9. 7. 2014 1:14)

Ešte mám jednu takú kacírsku otázku na všetkých. Čo si myslíte o použití

if(preg_match('/duplicate.+entry/i', $e->errorMessage)){...}

Nieje udržovanie aktuálnych kódov pre duplicate entry to isté ako udržovanie aktuálnych error mesidžov???

Spíš myslím, že radši ty kódy, přijde mi to takový jistější. Zkus znovu mrknout na to video o výjimkách, co je nahoře. Myslím, že tam se o tom taky mluvilo