Nette/Forms – addDatetime smazatelný?

My jsme to v našem rozšíření dělali tak, že jsme dědili TextBase kvůli nullable, akorát ve forms: 3.1.13 už to takhle snadno nepůjde.

díky, je to boží. Ještě addColor a můžu v aplikaci úplně vynechat setHtmlType :)

@DavidGrudl

Prosím doplnit do nových DateTimeControl prvků (addDate(), addDateTime(), addTime()) automatickou hodnotu setOption(‚type‘, ‚datetime‘). Všechny existující formulářové prvky mají hodnotu ‚type‘ automaticky nastavenu.

Já tato option používám pro nastavení custom css tříd ve formulářích:

$class = $component->getControl()->getAttribute('class');

$class .= $class ? ' ' : '';
$class .= match($component->getOption('type')){
    'text', => 'text-input block-input',
    'increment', => 'increment-input block-input',
    'datetime' => 'date-input block-input',               // toto option nyní chybí
    'textarea' => 'text-area-input block-input',
    'select' => 'select-input block-input',
    'checkbox' => 'checkbox',
    'radio' => 'radio',
    'file' => 'upload-input',
    'dialogButton' => 'submit-button',
    'button' => $component instanceof SubmitButton
        ? 'submit-button'
        : 'control-button',
    default => '',
};

@mbrecher doplním

@mbrecher předpokládám, že tím myslel, že nemůže přepsat metodu addDatetime() protože nevrací TextBase. Nicméně nullable se tohle téma stejně vůbec jinak netýká.

@DavidGrudl koukám že type=color nelze vlastně nulovat a nulová hodnota je černá, což je totální shit. Všechny ostatní html inputy jdou normálně mazat, ale tohle ne. Pro úplnou podporu výběru barvy, aby šlo mít i prázdnou hodnotu to asi nepůjde bohužel použít

addTime je super.
@DavidGrudl Šlo by plnit pomocí ActiveRow kde se na TIME používá DateInterval?

DateTimeImmutable nemuže mít den a měsíc 00, protože to je mimo povolené hodnoty (i ten rok 0 je nesmysl, ale to projde). Naopak s uložením 0000-01-01 17:53:00 do MySQL žádný problém nemám…

@DavidGrudl

DateTimeImmutable nemuže mít den a měsíc 00, protože to je mimo povolené hodnoty

aha to jsem nevěděl, díky hodnotě datumu ‚0000–01–01‘ lze addTime() uložit do MySQL TIME a nejde uložit do MySQL DATETIME, což je ideální řešení.

• addDate, addDatetime a addTime – vyzkoušeno a funguje super
• addColor – funguje jak má, ale bohužel nelze hodnotu mazat (což je default chování browserů), takže stále potřeba workaround

Ještě jsem procházel aplikaci a máme tam často tyto případy (jen taková idea co by se mi líbilo):

$form->addText('tel')->setHtmlType('tel');

// $form->addTel('tel') by bylo víc cool

a

$form->addText('recipients', 'E-mail příjemnce (nebo více oddělené čárkou)');
$form->setHtmlType('email');
$form->setHtmlAttribute('multiple');

// $form->addEmail() s možností parametru multiple
// a automatickou validací zda jsou zadány jen validní e-maily oddělené čárkou

Dá se tam dát pattern v HTML a na telefonech se například zobrazuje „volat“ a „speciální klávesnice na čísla bez písmen“, když máš on-focus.

Editoval Pavel Kravčík (3. 11. 2023 14:50)

@MarekBartoš

time → time
datetime → datetime / timestamp
date → date

Ano, použít v MySQL odpovídající typ pro datum a čas je ideální. Ale:

MySQL type TIME je v Nette Explorer problematicky podporován. ActiveRow z databázové knihovny vrací nikoliv Nette\Utils\DateTime, ale DateInterval. Ten nelze jednoduše vykreslit v šablonách latte:

{$row->datetime}  {* MySQL DATETIME - latte vykreslí *}

{$row->date}  {* MySQL DATE - latte vykreslí *}

{$row->time}  {* MySQL TIME - latte NEUMÍ vykreslit *}

{$row->time|date 'H:i'}  {* MySQL TIME - toto filtr latte také NEUMÍ *}

Jak zaznělo v této diskuzi, nově přidaný prvek addTime() také nedokázal načíst čas z objektu DateInterval, což @DavidGrudl bleskově opravil, ale ne tak, že by změnil v ActiveRow DateInterval na Nette\Utils\DateTime s datumem ‚0000–01–01‘, ale přidal do addTime() schopnost zpracovat i DateInterval.

Jenže situace jaká byla v addTime() je i v šablonách Latte (code samply výše). Pokud by někdo opravdu chtěl použít pro čas typ MySQL TIME, tak si bude muset pro latte udělat speciální filtr.

Udělat si pro MySQL TIME speciální latte filtr není složité, nevyřeší se tím ale přímý výpis bez filtru {$row->time}, což se často používá při psaní šablon – nejprve nahodím do šablony data, až potom doplním filtry.

Nebo použít pro čas MySQL DATETIME – také ne ideální. V budoucnu bychom mohli využívat metadata uložená v MySQL serveru jako support pro automatické generování formulářů. Když budou správně nastaveny datové typy, tak pro DATE použijeme ve formuláři prvek addDate(), pro DATETIME použijeme addDateTime() a pro TIME použijeme addTime().

Ideální je věci nekomplikovat a mít stejné API pro datum a čas jak ve formulářích, tak v databázi, tak i v databázových tabulkách – tedy používat pro čas MySQL typ TIME, ale zbavit se DateInterval-u v ActiveRow.

Ještě je zde možnost jít v Latte stejnou cestou jako v addTime() a pozměnit latte tak, aby si automaticky převedlo DateInterval na Nette\Utils\DateTime. To by řešilo skoro vše, ale pořád zde zůstane nekonzistentní DateInterval v ActiveRow pro sloupec TIME.

Přitom DateInterval není nic speciálně navrženého jenom pro čas, umožňuje vyjádřit časový interval i v letech a měsících, takže by nejlepší ho z Nette úplně vyřadit. Bohužel je to BC, ale čas se moc nepoužívá a většina lidí bezpochyby stejně pro čas používá DATETIME právě kvůli nemožnosti výpisu v latte.

Editoval m.brecher (3. 11. 2023 17:21)

evromalarkey napsal(a):

@DavidGrudl vesměs u všech input typů by měla být dobrá podpora v prohlížečích až na week a month

Jen na okraj: datetime-local má trochu nepříjemnou podporu ve Firefoxu, kde picker nabízí jen datum a čas se musí dopsat ručně. Jiné prohlížeče nabízí picker pro datum i pro čas.