Upravit chování validace formulářů?

S tím zcela souhlasím, ale zrovna takováto série chyb mi uživatelsky příjemnější nepřipadá. Tj. pokud zapomenu vyplnit věk, nevidím přínos v tom, že se vypíší hlášky ‚věk není číslo‘, ‚věk musí být větší než 0‘ apod. Naopak tím uživatele zmatu.

kravco napsal(a):

Ja som za, avšak zastaviť validáciu konkrétneho prvku, nie celého formulára.

Přesně tak a máš pravdu, že jsem to zapomněl zdůraznit, první post jsem doplnil.

dmajda napsal(a):

Validace a přetypování jsou IMO dvě různé věci. Přesněji řečeno, přetypování může zároveň obsahovat validaci, že uživatel zadal něco přetypovatelného, ale validace sama by neměla s hodnotou nic dělat.

Zkus se odprostit od zažitých představ, s nimi ten framework nevybudujeme :-))

Zapomeňme chvíli na filtry a validátory a vraťme se k nejprostšímu způsobu zpracování vstupních dat. Ono zpracování je řada úkonů, kde na počátku máme hodnotu zadanou uživatelem a na konci je buď chybová zpráva nebo nativní hodnota. Podstatné je, že úkony následují v daném pořádí a že řada je právě jedna (byť třeba rozvětvená pomocí podmínek).

V tom se věřím shodneme.

Zároveň z toho vyplývá důležitá věc: implementace pomocí dvou řad (filtry + validátory) je zcela špatná. Nelze říct, která řada se má provádět dřív. A lze vymyslet řadu jednoduchých příkladů, ze kterých je nesmyslnost této implementace hned patrná.

Řada, úkony. Je spíše terminologická záležitost, jestli „řadu“ budeme nazývat „validace“ a „úkony“ slovem „validátory“ (nebo třeba „pravidla“?). Můžeme ony úkony rozdělit do dvou typů, „validátory“ a „filtry“. Jeví se mi ale pragmatičtější je nerozdělovat. (Nebo je důvod je rozdělovat ten, že je rozdělují v nějakém jiném frameworku?)

Byl bych pro úplně novou metodu setType($type, $message = NULL), která by zařídila přidání odpovídající validace (tj. zavolala by addRule(Form::WHATEVER, $message) a následné přetypování hodnoty controlu v případě, že by byl validní. Tím by se ty dvě myšlenky od sebe odlišily.

Tady nejde jen o přetypování, ale jakoukoliv jinou konverzi, třeba na malá písmenka apod.

Určitě by to taky chtělo mít přístupnou hodnotu před přetypováním i po něm.

To spíš raději ne…

ad 1) Souhlas. Ale asi bych nechal moznost zapnout i soucasne chovani (vypis vsechn erroru prvku) i kdyz me ted nenapada zda to vubec nekdo nekdy vyuzije.

ad 2) Ja bych dany problem rozclenil na 2 veci:

• co chceme po uzivateli (pravidla)
• co za uzivatele udelame mi (prevod, zariznuti, nahrada)

Jinak napad zajimavy, ale urcite budou problem s kompatibilitou. Napr ted muze nekde testovat

if($form['prvek']->value != '') {

}

ale ted by musel testovat vuci null.

Pokud by se jadnalo napriklad jen o INT a FLOAT tak jsem urcite pro. Dale by bylo zajimave napriklad vstup pro datum, cas, datum a cas. Vystupem by mohl byt UnixTimeStamp nebo nejaky jiny dany format. Zajimave by bylo i aby uzivatel mohl definovat nejake svoje prevody.

Implementoval bych tedy následující:

1. po první validační chybě se proces zastaví, nebude-li na pravidlu určeno jinak (způsob určení není zatím zvolen – použít třeba čtvrtý parametr addRule()?)
2. přidám možnost zastavit proces kdykoliv; buď metodou addTerminate(), addTerminator() nebo terminate()
3. v důsledku této změny budu muset odstranit metodu addRuleFor(). Lze ji ovšem nahradit konstrukcí využívající addConditionOn(), viz příklad:

$form['send']->addCondition(Form::EQUAL, TRUE)
	->toggle('sendBox')
	->addRuleFor($form['city'], Form::FILLED, 'Enter your shipping address')
	->addRuleFor($form['country'], Form::FILLED, 'Select your country');

// se změní na

$form['send']->addCondition(Form::EQUAL, TRUE)
	->toggle('sendBox');

$form['city']->addConditionOn($form['send'], Form::EQUAL, TRUE)
	->addRule(Form::FILLED, 'Enter your shipping address');

$form['country']->addConditionOn($form['send'], Form::EQUAL, TRUE)
	->addRule(Form::FILLED, 'Select your country');

p.s. nahradit toggle() za addToggle()?

Jsem pro hybrida, najde uplatnění třeba při koverzi dat do tvaru pro databázi (volitelně vyplnitelná políčka nastavovat na NULL, pokud jsou prázdná, konvertovat datum do tvaru RRRR-MM-DD,…)

David Grudl wrote:

$form['send']->addCondition(Form::EQUAL, TRUE)
	->toggle('sendBox')
	->addRuleFor($form['city'], Form::FILLED, 'Enter your shipping address')
	->addRuleFor($form['country'], Form::FILLED, 'Select your country');

// se změní na

$form['send']->addCondition(Form::EQUAL, TRUE)
	->toggle('sendBox');

$form['city']->addConditionOn($form['send'], Form::EQUAL, TRUE)
	->addRule(Form::FILLED, 'Enter your shipping address');

$form['country']->addConditionOn($form['send'], Form::EQUAL, TRUE)
	->addRule(Form::FILLED, 'Select your country');

Len chcem reagovat na ten tvoj priklad, priamo s temou to nesuvisi. Riesim podobny formular, kde sa cast formularu skryva na zaklade toggle pre checkbox. A rovnako cez podmienku ak je checkbox checked su tie polia required. Ale ani po odoslani formulara a jeho neuspesnom validovani a znovuvykresleni nie su tie polia oznacene rendererom ako required (class=„required“).

Díky :-)

2. Hybrid mezi validátorem a filtrem

Nastal nějaký postup v této věci?

Je to poměrně častý problém: uživatel vyplňuje telefonní číslo s mezerama, rodné číslo s lomítkem apod.

Osobně to řeším filtováním znaků už při jejich zadávání:

$form->addtext('psc', 'PSČ:')
         ->addConditionOn($orderForm['fakturacni_udaje'], Form::EQUAL, TRUE)
              ->addRule(Form::FILLED, 'Zadejte PSČ.')
              ->addRule(Form::MAX_LENGTH, 'PSČ je příli dlouhé.', 5)
              ->addRule(Form::NUMERIC, 'PSČ není číslo.');
$form['psc']->getControlPrototype()
                 ->onkeydown = 'this.value = this.value.replace(" ", "");';