Bug v selectboxu ->setPrompt() nelze pouzit v nette 2.4

Ahoj,
já jsem to vyzkoušel, viz můj kód:

$countries = [
    'Europe' => [
        'CZ' => 'Česká Republika',
        'SK' => 'Slovensko',
        'GB' => 'Velká Británie',
    ],
    'CA' => 'Kanada',
    'US' => 'USA',
    '?'  => 'jiná',
];

$form = new Nette\Forms\Form;
$form->addSelect('country', 'Země:', $countries)
    ->setPrompt('Zvolte zemi');

$form->addSubmit('send');

if($form->isSubmitted()) {
	dump($form->getValues());
	exit;
}

echo $form;

A výstup mám podle očekávání, testováno na nette/forms v2.4.4

Nette\Utils\ArrayHash #58f4
  country => NULL

Jaký objekt na formuláře používáš? Nemáš nějaké své rozšíření?

Zkusil jsem

$form->addSelect('country', 'Země:', $countries)
    ->setPrompt('Zvolte zemi')
	->setDefaultValue('SK');

a pak toto

$form->addSelect('country', 'Země:', $countries)
    ->setPrompt('Zvolte zemi');

$form->setDefaults(['country' => 'SK']);

Chování je správné. Žádná exception a buď dostanu vybranou hodnotu, defaultní nebo NULL když vyberu ‚Zvolte zemi‘.

Ukaž implementaci.

Editoval h4kuna (20. 4. 2017 8:38)

@krucinal Mělo by fungovat $form['countries']->checkAllowedValues = FALSE.

setDefaultValue('') prostě nevolej, prompt je vybrany automaticky.

@krucinal Řešil jsem něco podobného (nastavuji do formuláře výchozí hodnotu z query parametrů, ale nedokáži snadno ověřit, že ta hodnota je dobře před nastavením) a nakonec jsem došel k následujícímu hacku (@DavidGrudl se teď možná trochu zděsí, co jsem vymyslel).

	private function trySetDefaultValue(BaseControl $control, $newValue): bool
	{
		assert(!$control->hasErrors(), 'errors may be lost by cleanErrors()');

		if ($newValue === NULL) {
			return FALSE;
		}

		$currentValue = $control->getValue();

		try {
			$control->setDefaultValue($newValue); // may throw
			$control->validate();

			if (!$control->hasErrors()) {
				return TRUE;
			}

		} catch (\Nette\InvalidArgumentException $e) {
			// suppress, probably not scalar or invalid value for select box
		}

		$control->cleanErrors();
		$control->setValue($currentValue);
		return FALSE;
	}

krucinal napsal(a):

@DavidGrudl ok, potom ale nemuzu generalne vlozit cela data odeslana formularem do setDefaults() a naplnit tak formular

Tomu vůbec nerozumím. Do setDefaults() se dávají výchozí zobrazené hodnoty, než je formulář odeslán. Jakmile je odeslán, setDefaults se vůbec neřeší. Viz https://doc.nette.org/cs/forms

ale musim si nejdriv zkontrolovat, jestli nahodou nemaji data selectboxu hodnotu promptu

Co je to hodnota promptu? Prompt neber vůbec jako hodnotu.

Vychazi mi z toho, ze nejlepsi bude tuhle featuru radsi nepouzivat a napsat si to po svem a predelat ted stovky formularu v projektu a misto promptu z nette si vsude rucne pridat tu prvni volbu do pole dat :-(

To určitě není potřeba, jen myslím používáš formulář nějak jinak.

Tak na to by určitě bylo lepší udělat nějakou BaseControl::isValueAllowed(), ne?

Ano, to by bylo mnohem lepší.

Ted uz jen ciste akademicky na vysvetleni, protoze jsem mozna ne uplne presne napsal o co mi slo.

Mozna pouzivam formular nezvykle, ale me se to jevi uplne normalni. Jen vyuzivam nette k vygenerovani/naplneni a zprocesovani formulare bez nejake vyssi logiky. Jedna se v tomto pripade o formular na vyhledavani (teoreticky se ale muze jednat o libovolny formular), ktery muze fungovat tak, ze sam neco odesle (pak setDefaults nejspis nepotrebuju), ale zaroven na stranku muze vest odkaz odjinud, kdy uz muzou existovat fragmenty dat v url (nebo jinem zdroji), potom je chci pouzit jako vychozi hodnoty.

Nejjednodussi co me napada je proste zavolat setDefaults generalne tam kde buduju formular a nemuset u toho neco hackovat/podminkovat.

Fakt, ze hodnota promptu nesmi byt odesilana je pro me nova informace, celou dobu jsem to bral jako nastroj pro prvni hodnotu selectboxu. Mam nekde nejake metody, ktere ziskavaji data pro selectbox, napr. z ciselniku. Ty ale vraci jen data, prvni polozka je veci formulare, jelikoz nekdy ji tam potrebuju, jindy ne, pokazde muze mit jiny text, specificky pro dany fomrular a pokazde muze mit jiny logicky vyznam.
Na tohle se mi prompt naramne hodil, nicene uz se nyni pouzit neda.

Vypnuti kontroly hodnot je sice reseni, ale nelibi se mi. Ta vestavena kontrola je vyborna vec.

Priklad:

protected function createComponentFilterForm()
    {
        ...
        ...
        $filterForm = new Form();

        $filterForm->setMethod(Form::GET);
        ....
        $filterForm->addSelect('city', 'Město', $tCities->getCityListPreferred())
            ->setPrompt('--- všechna města ---')
        ;
        ...
        ...
        $filterForm->addSubmit('submit', 'Odeslat');

        $filterForm->setDefaults($this->getParameters());

        $filterForm->onSuccess[] = [$this, 'filterFormSubmitted'];

        return $filterForm;
    }

    public function filterFormSubmitted() {
        ...
    }

Už tomu rozumím.

Prompt v selectboxu není žádná hodnota, je to jen „popiska“. Pokud je prvek povinný, vybrat nejde, pokud je nepovinný, getValue() vrací NULL, jak píše @F.Vesely. Prázdný řetězec je nějaká interní nepodstatná záležitost, od které jsi odstíněný.

No tedy jen do doby, než sáhneš pod kapotu, do hrubých dat, což je právě $this->getParameters(). Ty je potřeba očistit, zvalidovat. Takže třeba '' nahradit za NULL, ale taktéž ověřit, že jsou stringy tam kde mají být stringy, pole kde mají být pole atd.

Fakt je, že tohle by mohl formulář umět, přijmout hrubá data, ale v tuto chvíli snadno neumí.