Formular – addCheckboxList – prazdne values v onSuccess

@Creator13

Takisto mam problem s touto kontrolou. Funguje mi az ked zadam ako podmienku 2. Ak neoznacim ziadny checkbox, tak mi tuto podmienku neberie do uvahy.

Podmínky přidané addRule() se validují až když prvek obsahuje nějaká data. Aby Ti validace fungovala i pro prázdný prvek, musíš přidat setRequired():

$form->addCheckboxList('products', 'Produkty', $checkboxListData)
			->setHtmlAttribute('class', 'form-check-input w20 h20 order-create-control-selected-product')
			->setRequired('Musíte vybrať aspoň 1 položku')
			->addRule($form::MinLength, 'Musíte vybrať aspoň %d položku', 1);

V onSuccess dostavam prazdne $values. Ked pozriem $form->getHttpData(), tak sa tam pole s oznacenymi checkboxami nachadza.

Nevalidní data formulář do $values nepustí – může to mít souvislost s chybějícím setRequired() jak jsem uvedl výše, oprav validaci a uvidíš.

Editoval m.brecher (23. 8. 13:11)

Podarilo sa mi to vyriesit

Po odoslani formulara mi nadradena komponenta predavala prazdne pole $products

public function create(array $products): Form

Nadradena komponenta, z ktorej form berie $products.
Presunul som z render() do __construct

class CreatePrintOrderStep1Control extends Control
{

	use OrderActionPanelControlTrait;
	use PrintOrderCreateStep1FormControlTrait;

	private ?int $userId = null;

	/**
	 * @var array<int, OrderProductEntity>
	 */
	private array $products = [];

	public function __construct(
		private readonly UserDataProvider                        $userDataProvider,
		private readonly OrderActionPanelControlFactory          $orderActionPanelControlFactory,
		private readonly OrderProductDataProvider                $orderProductDataProvider,
		private readonly PrintTechnologyDataProvider             $printTechnologyDataProvider,
		private readonly ArrayService                            $arrayService,
		private readonly PrintOrderCreateStep1FormControlFactory $printOrderCreateStep1FormControlFactory,
	)
	{
		$this->initialize();
	}

	public function initialize(): void
	{
		$userId = $this->userDataProvider->getUserId();
		$products = $this->orderProductDataProvider->getProductsForCreatingPrintOrder($userId, true);

		//products for form
		$this->products = $products;
	}

	public function render(): void
	{
		$this->getTemplate()->products = $this->products;
		$this->getTemplate()->setFile(__DIR__ . '/templates/createPrintOrderStep1.latte');
		$this->getTemplate()->render();
	}

Je toto riesenie korektne? Ma komponenta ine moznosti pre inicializaciu a kontrolu, napr. ako ma presenter startup, action?

m.brecher napsal(a):
Úplně čisté řešení to není, protože v konstruktoru by bylo lepší inicializaci nemít

Proč? Konstruktor je k tomu, aby inicializoval objekt se vším, co ke své existenci potřebuje.

O něco čistší než dávat inicializaci komponenty do konstruktoru komponenty je volat inicializaci v metodě createComponent<*>()

Někdy asi ano, pokud komponenta vykresluje obecná data, která pochází z venku (i když pak bych spíš očekával setter).
Pokud ale komponenta vykresluje data dodaná modelem (z jednoho místa), nedává mi smysl opakovat se s každým createComponent(), když to může být v konstruktoru na jednom místě.

Dejme tomu, že máš 2 akce a 4 handle. Nyní se při všech 6 akcích provede dotaz do databáze (v 5 případech zbytečně, možná někdy ve všech 6 zbytečně) i když vůbec nesaháš na komponentu. Můžeš taky udělat lazy getter:

private function getProducts() {
	return $this->products ??= $this->orderProductDataProvider->getProductsForCreatingPrintOrder($userId, true);
}

a ušetříš dotazy do databáze. Pokud chceš jít cestou Konstruktor je k tomu, aby inicializoval objekt se vším, co ke své existenci potřebuje. Tak products předávej z presenteru a udělej factory na komponentu, jen on ví, kdy se komponenta opravdu použije.

Martk napsal(a):

Dejme tomu, že máš 2 akce a 4 handle. Nyní se při všech 6 akcích provede dotaz do databáze (v 5 případech zbytečně, možná někdy ve všech 6 zbytečně) i když vůbec nesaháš na komponentu.

Pokud je $products nepovinné pro fungování komponenty, tak bych to do ní cpal setterem a tahal getterem (zda lazy nebo ne není aktuálně předmětem diskuze). Což jsem ale popsal už prvně.

Pokud je $products nevyhnutelně potřeba pro existenci komponenty, připadá mi legitimní chtít to v konstruktoru a instancování objektu bez toho ani nepustit.

Tak products předávej z presenteru a udělej factory na komponentu

Někdy ano, pokud jsou data závislá ještě na něčem z presenteru. Ale pokud jsou fixně tahána z modelu, není důvod mít to na 10 místech (pokud komponentu používám 10x), to už jsem ale také psal prvně.

jen on ví, kdy se komponenta opravdu použije.

Komponenta to ví zrovna tak.

Editoval Kamil Valenta (6. 9. 8:18)

@Martk to je ale iný prípad, diskusia sa týkala inicializácie komponenty v konstruktore… tvoj use-case samozrejme je logicky správny, a je možné ho zrealizovať tak, že si tú kolekciu produktov vyžiadam v konstruktore… pretože ak je komponenta na nich závislá, konstruktor je jediné správne miesto, kde si ich vypýtať