Form, Database Explorer, prosím o radu

Šaman napsal(a):

MW napsal(a):

Ptám se především proto, že se mi nejak nezdá, jestli je dobře ten setDefaults()->fetch()->, jinak jsem to tam nedostal a trochu nechápu, proč, když to vrací následující prvek… podle doc..

Vždyť nevoláš $form->setDefaults()->fetch(), to by opravdu nedávalo smysl.
Ty voláš $form->setDefaults( $db->query()->fetch() ). Fetch je v tomto případě potřeba, protože do setDefaults máš předat pole dat, nikoliv aktivní databázový dotaz. (Resp. ty nad db nevoláš query, ale to je jedno, obecně přistupuješ k datům a pak si fetchneš výsledek.)

Navázat obsluhu update v presenteru je ok. Záleží jakou máš strukturu aplikace. Já mám formuláře v modelu, ale používám orm, takže formulář je vedle entity, kterou upravuje. Základní obsluha je pak také navázaná v tom modelu, ale minimálně redirect si má řešit až ten presenter.

Moc díky! Jen ten fetch() není asi úplně šťastně popsán v DOC… nebo jsem to přehlédl.

Editoval MW (27. 6. 21:45)

@Šaman

Já mám formuláře v modelu, .....

To mě přijde jako zajímavý koncept.

V jaké formě máš tyto formuláře vytvořené ?? Jako traity, které obsahují metodu createComponent<Form>(), nebo jako plnohodnotnou třídu poděděnou z UI\Control, která “obaluje” vlastní Form ??

Poznámka: @KamilValenta používá opačný koncept než ty, kdy umisťuje modelové třídy do složky presenteru a má tak pohromadě vše co presenter používá – formuláře, modelové třídy, šablony a samotný presenter.

@MW

$this->settings->getSettingsTable()->fetch()

Metoda fetch() knihovny Explorer se dá použít pro čtení řádků z objektu Selection v cyklu např.:

$articles = $database->table('article');
while($article = $articles->fetch()){
    echo $article->title;
}

nebo mimo cyklus dává smysl pokud vybíráme podle sloupce, který má unikátní index a je tak zaručeno, že výběr z tabulky Selection obsahuje pouze jeden řádek:

  $article = $database->table('article')->where('title', $title)->fetch();  // sloupec title má UNIQUE index

Pro práci s jedním řádkem databázové tabulky ve formuláři nejčastěji používá výběr pomocí metody get($id), nebo where(…) uvedený výše.

Když si rozebereme Tvoje dotazy do databáze:

$this->settings->getSettingsTable()->fetch()

$this->settings->getSettingsTable()->update($form->values)

Tak mě tam chybí výběr řádku z tabulky nějakým parametrem, který by se měl ve formuláři aktualizovat, – třeba $id. Otázkou je, co se skrývá za getSettingsTable(), jestli tam není ten parametr schovaný.

MW napsal(a):

protected function createComponentNastaveniForm(): Form {

	$form = $this->nastaveniFactory->create();
	$form->setDefaults($this->settings->getSettingsTable()->fetch());

	$form->onSuccess[] = function ($form) {

	$this->settings->getSettingsTable()->update($form->values);

	$this->flashMessage("Nastavení uloženo.");
	$this->redirect('Nastaveni:');

	};
	return $form;
    }

Mně by tady vadilo, že se v presenteru volá update() přímo nad selectionou. Metoda presenteru tedy sahá přímo do db bez vědomí modelu, což není dobře (projeví se, až přibude požadavek na přístupová práva, logování změn apod.)
Za mě by bylo lépe, kdyby přímo modelová třída settings měla svou metodu update().
Tedy:

$form->onSuccess[] = function ($form) {
$this->settings->update($form->values);
$this->flashMessage("Nastavení uloženo.");

V počátku ta metoda klidně jen vše přenese do $this->getSettingsTable()->update(), ale když se později něco změní, upravíš jen update() v settings.