Lze použít vícekrát onSuccess[]?

V editaci většinou předvyplníš formulář výchozími hodnotami.
Takže jestli jde o editaci, nebo vložení poznáš už ve formuláři a to tak, že pokud tam pošleš výchozí data, tak je to editace, pokud ne, tak je to přidání. Nepotřebuješ 2 onSuccess metody.

Tohle řeším tak, že data z formuláře v success metodě pošlu do modelu a tam provedu zpracování. Pokud existuje například hodnota id, tak provedu UPDATE, jinak INSERT.

Michal Kumžák napsal(a):

Tohle řeším tak, že data z formuláře v success metodě pošlu do modelu a tam provedu zpracování. Pokud existuje například hodnota id, tak provedu UPDATE, jinak INSERT.

Děkuji, ukázal byste mi prosím nějaký příklad?

FormFactory:

class FormFactory
{
	public function __construct(
		private Repository $repository,
	) { }

	public function create(int $id): Form
	{
		$form = new Form();
		$form->addHidden('id');
		$form->addText('content', 'Obsah zprávy')
			->setRequired();

		$form->setDefaults($this->repository->get($id) ?? []);
		$form->addSubmit('submit', 'Odeslat');

		$form->onSuccess[] = [$this, 'onSuccess'];
		return $form;
	}

	public function onSuccess(Form $form): void
	{
		try {
			$this->repository->save($form->getValues(true));
		} catch (Exception $e) {
			$form->addError('Vyskytla se chyba.');
		}
	}
}

Repository:

class Repository
{
	public function __construct(
		private Explorer $db,
	) { }

	public function save(array $values): void
	{
		$sel = $this->db->table('repo');
		$id = $values['id'];
		unset($values['id']);
		if ($id) {
			$sel->get($id)->update($values);
		} else {
			$sel->insert($values);
		}
	}

	public function get(int $id): ?ActiveRow
	{
		return $this->db->table('repo')
			->get($id);		// Nula v MySQL vrátí nic, jelikož se indexuje od 1
	}
}

Presenter:

class Presenter extends UiPresenter
{
	#[Inject] public FormFactory $formFactory;
	private int $id;

	public function actionAdd(): void {
		$this->id = 0;
	}

	public function actionEdit(int $id): void {
		$this->id = $id;
	}

	public function createComponentForm(): Form
	{
		$form = $this->formFactory->create($this->id);
		$form->onSuccess[] = [$this, 'onSuccess'];
		return $form;
	}

	public function onSuccess(): void
	{
		$this->flashMessage('Úspěšně uloženo', 'success');
		$this->redirect('this');
	}
}

Upozorňuju, že copypasta není dobrý nápad. Jde o příklad. Není zde ošetřeno práva k signálům a ani to není úplně všechno čistě napsané. Doporučuju projít třeba kurz, kde je to všechno ukázané i se zabezpečením.

Editoval Polki (15. 10. 2021 18:27)

m.brecher napsal(a):

Jeden formulář, který by uměl „VŠECHNO“ – založit nový, editovat a smazat záznam je potřeba dost často. Já to řeším jedním formulářem a jednou akcí v presenteru pro všechno.

To není příliš šikovný přístup. Mnohem lepší je mít pro formulář samostatnou komponentu a v presenteru mít klasicky tři akce add, edit, delete.

presenter

<?php
declare(strict_types=1);

namespace App\AdminModule\CiselnikyModule\Presenters;

use Nette,
	App\Model,
	App\Presenters;

class MenyPresenter extends Presenters\BasePresenter
{

	/** @var Model\KurzyModel */
	private $kurzyModel;

	/** @var \IMenyControlFactory @inject */
	public $menyControlFactory;

	function __construct(Model\KurzyModel $kurzyModel) {
		parent::__construct();
		$this->kurzyModel = $kurzyModel;
    }

	public function renderDefault($page='') {
		if (!$this->user->isAllowed('ciselniky', 'read')) {
			$this->error('Nemáte oprávění na tuto akci', 403);
		}
		$this->template->meny = $this->kurzyModel->getMeny();
	}

	public function renderAdd() {
		if (!$this->user->isAllowed('ciselniky', 'add')) {
			$this->error('Nemáte oprávění na tuto akci', 403);
		}
	}

	public function renderEdit($id) {
		if (!$this->user->isAllowed('ciselniky', 'update')) {
			$this->error('Nemáte oprávění na tuto akci', 403);
		}
	}

	public function actionDelete($id) {
		if (!$this->user->isAllowed('ciselniky', 'delete')) {
			$this->error('Nemáte oprávění na tuto akci', 403);
		}
		$rows = $this->kurzyModel->deleteMena($id);
		if ($rows > 0) {
			$this->flashMessage('Měna byla smazána.');
		} else {
			$this->flashMessage('Měna nejde smazat, je používána.');
		}
		$this->redirect('default');
	}

	protected function createComponentKurzy() {
		$control = $this->menyControlFactory->create();
		$control->setId($this->getParameter('id'));
		$control->onFormSave[] = function (\MenyControl $control) {
			$this->redirect('default');
		};

		return $control;
	}
}

latte pro akci add a edit může být stejné

{block content}
<h1>Nový kurz / Editace kurzu</h1>

{control kurzy}

control

<?php
declare(strict_types=1);

use Nette\Application\UI;

class KurzyControl extends UI\Control
{
	/** @var App\Model\KurzyModel */
	private $kurzyModel;

	public $onFormSave;

	private $id;

	public function __construct(App\Model\KurzyModel $kurzyModel) {
		$this->kurzyModel = $kurzyModel;
	}

	public function setId($id) {
		$this->id = $id;
	}

	public function render() {
		$this->template->setFile(__DIR__ . '/kurzy.latte');
		$this->template->id = $this->id;
		$this->template->render();
	}

	/**
	 * @return UI\Form
	 */
	public function createComponentKurzy() {
		$form = new UI\Form;
		$form->setRenderer(new AlesWita\FormRenderer\BootstrapV4Renderer);

		$form->addSelect('mena_id', 'Měna:', $this->kurzyModel->getMenyArray(false, 'CZK'))
			->setHtmlAttribute('title', 'Prosím vyberte měnu')
			->setRequired('Prosím vuberte měnu.');

		$form->addText('kurz', 'Kurz:')
			->setHtmlType('number')
			->setRequired('Prosím zadejte kurz.');

		$form->addText('platnost_od', 'Platnost od:')
			->setHtmlType('date')
			->setRequired('Prosím zadejte platnost od.');

		$form->addSubmit('send', 'Uložit');
		$form->addSubmit('zpet', 'Zpět')
			->setValidationScope([]);

		$form->addHidden('id');

		$form->setDefaults($this->kurzyModel->getKurzById($this->id));

		$form->onSuccess[] = [$this, 'kurzySucceeded'];
		return $form;
	}

	public function kurzySucceeded($form, $values) {
		if ($form['send']->isSubmittedBy()) {
			$this->kurzyModel->saveKurz($values);
		}
		$this->onFormSave($this);
	}
}

kurzy.latte

{control kurzy}

a ještě interface továrny

<?php
declare(strict_types=1);

interface IKurzyControlFactory
{
	/** @return KurzyControl */
	public function create();
}

Jo a když by nepotřeboval dělat kontrolu práv, tak renderAdd a renderEdit ani nemusí v presenteru být, stačí mít add.latte a edit.latte

Editoval Michal Kumžák (17. 10. 2021 17:55)

Polki napsal(a):

Spravne je to jak pise @MichalKumžák
Ohledne zjistovani existence neceho v route bych byl opatrny muze se snadno stat, ze budes mit spousty zbytecnych dotazu do db.
Taky to, ze se to bude rozhodovat az v presenteru nema vliv na to jaky error se vrati

@Polki

V Routě ověřuji jenom formát parametru id – samozřejmě bez dotazů do databáze, tomu je lepší se v Routeru vyhnout. Id musí být integer vyjma 0. Pokud ne tak Router vyhodí 404.

Pokud Router povolí jakékoliv id, třeba i „abc“, tak v presenteru, kde se předává id jako parametr do akce takto:

declare(strict_types=1);

.........

public function actionEdit(?int $id = null)
{
.....
}

dojde k vyhození TypeError, protože id = „abc“ neproleze typehintem v metodě akce. Výsledkem by byla nepěkná 500 v situaci, kdy server nemá žádnou vnitřní chybu, ale klient pouze poslal neplatné url. Ale není to moc důležité, protože:

Přímo v presenteru jde také vyhodit 404 pro id = „abc“, museli bychom ale upustit od typování předávaného parametru do akce:

declare(strict_types=1);

.........

public function actionEdit($id = null)		// nyní může být id 1,2,3 ale i abc ;)
{
.....
}

model by pro id=„abc“ vrátil null a 404 by vyhodil presenter – tak jako tak se musí existence záznamu ověřovat.

@mbrecher
BTW v tvém případě pokud budeš chtít nastavit pro akce ADD, EDIT a DELETE různá práva, tak to bude obtížné.
Navíc pokud budeš provádět DELETE, tak se ta ze startu bude chovat jako EDIT, což znamená, že provede natažení prvku z databáze, vytvoření formuláře, vyhodnocení formuláře, zavolání všech callbacků navěšených na formuláři a až nakonec nějaký SuccessCallback, který ještě bude špatně ifem rozdělený na 2 rozdílné funkčnosti (INSERT/EDIT a DELETE) – Říká ti něco princip SingleResponsibility?
A to nemluvím o situacích, kdy chceš mít tlačítko na smazání například ve výpise článků v adminu, kde se jinak formulář vůbec nepoužívá. Najednou budeš mít buď zdvojenou logiku pro mazání, nebo budeš používat formulář na něco, na co se používat nemá.

Přitom pro delete úplně stačí například toto:

public function handleDelete(int $id): void	// nebo actionDelete...
{
	if(!$this->user->isAllowed('Item', 'Delete')) {
		// Not allowed error nebo 404 pokud nechceš aby uživatelé, kteří nemají přístup nevěděli, jakou používáš URL pro mazání
	}
	if (!$this->repository->delete($id)) {
		// 404 záznam neexistuje
	}
	$this->redrawControl('ItemsTable'); // Nebo přesměrování, pokud nejsi ve výpise, ale v detailu.
}

A v repu klasicky:

public function delete(int $id): int
{
	return $this->db->table('items')		// pokud neexistuje záznam s daným ID, tak jej nesmaže a vrátí nulu, takže v presenteru se to vyhodnotí jako true a vypíše se 404 error záznam nenalezen
		->where('id', $id)
		->delete();
}

Tento způsob zápisu udělá 1 dotaz do db, jsou ošéfena práva a hlavně nebudeš zbytečně tvořit tunu instancí tříd formulářových prvků, komponent apod, které se stejně nevyužijí.

Proto je best mít to jak psal @MichalKumžák

Editoval Polki (17. 10. 2021 21:03)

Polki napsal(a):

@mbrecher
BTW v tvém případě pokud budeš chtít nastavit pro akce ADD, EDIT a DELETE různá práva, tak to bude obtížné.
Navíc pokud budeš provádět DELETE, tak se ta ze startu bude chovat jako EDIT, což znamená, že provede natažení prvku z databáze, vytvoření formuláře, vyhodnocení formuláře, zavolání všech callbacků navěšených na formuláři a až nakonec nějaký SuccessCallback, který ještě bude špatně ifem rozdělený na 2 rozdílné funkčnosti (INSERT/EDIT a DELETE) – Říká ti něco princip SingleResponsibility?
A to nemluvím o situacích, kdy chceš mít tlačítko na smazání například ve výpise článků v adminu, kde se jinak formulář vůbec nepoužívá. Najednou budeš mít buď zdvojenou logiku pro mazání, nebo budeš používat formulář na něco, na co se používat nemá.

@Polki máš to velmi dobře promyšlené z pohledu praxe a velmi jednoduché kódy. Nechci s Tebou polemizovat, že jak to navrhuješ to bezpochyby bude dobře fachat. Ale:

a ) dvě rozdílné funkčnosti v handleru pro INSERT/EDIT s ifem, který by neměl být.

INSERT/EDIT provádí formulář na úrovni signálu a akce presenteru je tam nepodstatná. Pokud bych chtěl oddělit obsluhu INSERT/EDIT do samostatných callbacků, tak bych to řešil dvěma různými tlačítky pro INSERT/EDIT, na které bych navěsil samostatné callbacky. Vložit jeden a ten samý formulář pro INSERT/EDIT do různých akcí – tím oddělení callbacků nevyřešíme.

b ) Chování DELETE jako EDIT

Formulář s editačním a mazacím tlačítkem je záměrně vícefunkční komponenta, protože pro uživatele je to takhle ideální koncept. Zobrazit si detail entity a mít po ruce všechny operace které se s ní dají udělat. Ale je to jak píšeš – pokud uživatel neprovede editaci, ale mazání, tak byla příprava na editaci „zbytečná“.

c ) Možná bychom to všechno mohli někdy probrat v hospodě u sklenky oroseného…

@mbrecher

a ) dvě rozdílné funkčnosti v handleru pro INSERT/EDIT s ifem, který by neměl být.

INSERT/EDIT provádí formulář na úrovni signálu a akce presenteru je tam nepodstatná. Pokud bych chtěl oddělit obsluhu INSERT/EDIT do > samostatných callbacků, tak bych to řešil dvěma různými tlačítky pro INSERT/EDIT, na které bych navěsil samostatné callbacky. Vložit > jeden a ten samý formulář pro INSERT/EDIT do různých akcí – tím oddělení callbacků nevyřešíme.

Ne funkčnosti, ale práva.
Obecně například komentáře může přidávat kdokoliv přihlášený, tedy práva jsou $this->user->isAllowed('comment', 'insert')
a editovat můžeš jen svoje, pokud nejsi admin, takže práva vypadají takto: $this->user->isAllowed($comment, 'edit')
Pokud by jsi to měl v 1 metodě a je jedno co za kód to je, tak by vypadal asi nějak takto:

#[Inject] public CommentRepository $commentRepo;
private ?Comment $comment = null;
public function actionModify(?int $id = null): void // modify v tomto případě označuje edit/add
{
	// ADD START
	if ($id === null) {
		if (!$this->isAllowed('comment', 'add')) {
			$this->error('Nemáte práva přiávat komenty přihlašte se', 401);
		}
		return;
	}
	// ADD END

	// EDIT START
	$this->comment = $this->commentRepo->get($id);
	if (!$this->comment) {
		$this->error('Komentář nenalezen', 404);
	}
	if (!$this->isAllowed($this->comment, 'edit')) {
		$this->error('Nemáte právo editovat tento komentář.', 401);
	}
	// EDIT END
}

vs správně

#[Inject] public CommentRepository $commentRepo;
private ?Comment $comment = null;

public function actionAdd(): void
{
	if (!$this->isAllowed('comment', 'add')) {
		$this->error('Nemáte práva přiávat komenty přihlašte se', 401);
	}
}

public function actionEdit(int $id): void
{
	$this->comment = $this->commentRepo->get($id);
	if (!$this->comment) {
		$this->error('Komentář nenalezen', 404);
	}
	if (!$this->isAllowed($this->comment, 'edit')) {
		$this->error('Nemáte právo editovat tento komentář.', 401);
	}
}

Tak vidíš, že první případ najednou dělá if mezi 2 různými kódy, i když zbytek aplikace se chová pořád stejně. Máš jeden formulář na přidání i editaci.

Co je ještě důležité je to, že pokud budeš chtít aby uživatel viděl URL ve formátu
/komentar/pridat
/komentar/editovat/5
a napíšeš router takto (což už jsem hodněkrát viděl):

$router->addRoute('komentar/pridat', 'Comment:modify');
$router->addRoute('komentar/editovat/<id>', 'Comment:modify');

Tak přes tuto URL se dostanu do editace, což není úplně ok:
/komentar/pridat?id=5
Stejně tak tato URL vyhodí Error, což by neměla, protože parametr ID by se v přidání měl úplně ignorovat:
/komentar/pridat?id=-3

Takže cpát akce dokupy je špatná praktika.

b ) Chování DELETE jako EDIT

Nějak tomu nerozumím. Mazací tlačítko klidně můžeš dát na editační stránku tak jak to máš teď, ale místo na tvorbu a vyhodnocování celého formuláře to bude prostě jen nastylovaný odkaz, který bude odkazovat na mazací handle/action, jak jsem psal výše. Pro uživatele se funkčnost nezmění, jen ty budeš mít přepsáním lepší kód.

c ) Možná bychom to všechno mohli někdy probrat v hospodě u sklenky oroseného…

Bezva nápad. Akorát nemám moc možností cestovat. :D Práce mi zabírá téměř veškerý volný čas. No a jelikož já jsem od Ostravy a ty z Prahy, tak to moc růžově nevidím.

No a k tvojí reakci na Michala bych napsal asi jen to, že mu nešlo o to napsat sem dokonalý kód, který ukáže top, jak má vypadat aplikace zabezpečeně, ale jen zhruba nástřel, jak se používají komponenty.
Pokud chceš o komponentách něco víc vědět, tak pak buď článek na Planette, nebo kurz pokročilých na learn2code, případně školení od Davida, jestli tam má něco na formuláře/komponenty.

Editoval Polki (18. 10. 2021 13:43)

Jen poznámka: není dobré metody pro obsluhu formuláře nazývat handle***, protože to má v Nette speciální význam.