Forms\Controls\SelectBox by neměl vracet NULL

Možná jsem se nevyjádřil úplně přesně, nejde mi o to, aby bylo defaultně zapnuté setRequired(), ale aby základní chování převzalo vlastnost, která se uplatňuje při zapnutém setRequired(). Pokud bude nastaveno setPromt(), tak to už je z hlediska bezpečnosti vybraná položka, s tím už programátor počítá a to lze považovat za regulérně vybranou položku. setRequired() už nad tím pouze zajišťuje business logiku (musí být vybraná specifická položka, nebo lze odeslat s promt hodnotou?) a toho se můj požadavek netýká.

Jaký je problém s translatorem? Toho se to přece nijak netýká.

Přehnané hlídání programátora? Ty někdy potřebuješ, aby uživatel mohl pomocí FireBugu odeslat z formuláře hodnotu, která není na výběr? Pokud ne, tak bys správně měl všude ošetřovat, že nedostáváš NULL. Opravdu to děláš všude? A není to trochu zbytečné, když se o to může starat framework?

Ondřej Mirtes: Přesně tak.

A je opravdu potřeba nastavovat vlastní chybová zpráva na událost, ke které nemůže dojít jinak, než že se někdo úmyslně snaží hacknout aplikaci? Podle mě by bohatě stačilo, kdyby byla zpráva přepsatelná v $defaultMessages, takže bude společná pro všechny SelectBoxy v aplikaci.

jtousek: Ale tohle není problém, tak v tom případě tu zprávu bude vyžadovat jako parametr konstruktoru a pak ji předá přímo tomu vytvářenému Rule. Ale přijde mi zbytečné chtít po programátorovi, aby u každého SelectBoxu vypisoval nějakou zprávu, která se stejně nebude v žádném regulerním případě vypisovat.

Tak to si asi nerozumíme, já nechci aby bylo vždy zapnuté přesné setRequired(), je regulérní požadavek chtít nepovinný SelectBox, kde může zůstat vybraný Prompt (například filtrovací formulář). To co chci je, aby měl programátor vždy zajištěno, že se mu vrátí pouze hodnota, která je v SelectBoxu (nebo NULL, pokud má SelectBox prompt).

Tedy:

new SelectBox('a', array('first', 'second')); // vždy vrátí 0 nebo 1
new SelectBox('b', array('a' => 'first', 'b' => 'second'))->setPrompt(); // vždy vrátí a, nebo b
new SelectBox('c', array('first', 'b' => 'second'))->setPrompt('select'); // vždy vrátí 0, b, nebo NULL
new SelectBox('d', array('first', 'b' => 'second'))->setPrompt('select')->setRequired(); // vždy vrátí 0, nebo b

V současné implementaci je problém, že:

new SelectBox('a', array('first', 'second')); // může vrátit NULL

Ano, to je pravda, stejně tak může být deploynuta nová verze aplikace před odesláním formuláře. Ale z mého pohledu jsou to natolik řídké jevy, že bych stejně nevymýšlel nějakou speciální hláškou pro každý SelectBox a nechal bych tam defaultní (také co bys tam chtěl vůbec psát?). Ale proti gustu, klidně může přibýt do konstruktoru SelectBoxu nepovinný parametr pro tuto chybovou zprávu, pokud si ji někdo chce měnit pro každý SelectBox zvlášť.

Ale hlavní je, aby to Nette prostě hlídalo. V tom případě, který píšeš se ti vrátí NULL a aplikace někam uloží špatná data.

To bezesporu, ale když už formulář nabízí validování, tak by to měl dělat pořádně. Myslím si, že je v pořádku na základě pravidel v modelu vygenerovat formulář a počítat s tím, že data, která mi vrátí jsou validní. Pokud bys chtěl validovat jenom v modelu, tak můžeme celé formuláře zahodit a posílat $_GET , nebo $_POST rovnou do modelu, což určitě nechceme.

Jojo.

A ještě bych popřemýšlel, jestli by nebylo lepší, aby SelectBox, který má Prompt a je odeslán s neplatnou hodnotou nebyl také spíše označený jako invalidní místo toho aby vrátil NULL. Protože tohle se stane např. když uživatel odešle SelectBox a mezitím deployneme novou verzi aplikace se změněným číselníkem a uživatel odešle úplně jinak vyplněný formulář než chtěl. Jako menší zlo by mi připadalo ho nechat odeslat formulář znovu.