User Warning: Form was submitted but there are no associated handlers

Upozornění je to fajn, ale myslím že začátečník si bude pořád říkat, co má být ten associated handler a bude muset sám zjistit, jak problém vyřešit. Moc by se mi líbilo, kdyby Nette v chybách vysvětlovalo jaká událost nastala, co je v ná za problém a jak ji vyřešit. Momentálně je třeba si řešení domyslet.

Context: Form with ID product-form was submitted.
Problem: Form has no handler to process it on success.
Solution: Add callback to Form->onSuccess[] or Form->onSubmit[] to process sent data.

Editoval Marek Bartoš (29. 8. 2021 15:01)

Hm i když jsem začátečník od roku 2014, tak se mi zrovna minulý týden podařilo přesně tuhle botu udělat.
Konkrétně jsem měl strukturu takovou, že jsem dělal formulář jako komponentu a tedy jsem měl více navěšených onSuccess metod.
Jednu, která se v komponentě starala o zpracování dat a jednu, která se v Presenteru starala o přesměrování po úspěšném dokončení.

Problém byl, že jsem navěštil tu událost z Presenteru, ale zapomněl jsem navěštit zpracování dat, takže redirect s flashMessage proběhl v pohodě, ale data se nezpracovala. Divil jsem se a docela dlouho na můj vkus jsem hledal kde je chyba a proč se mi nevolá update nad databází.

Když jsem si všiml, že mi tam chybí navěšení události, tak jsem se sám divil, že po takové době, kdy jsou formy denní chleba jsem udělal takovou chybu. A to, že to formuláře tvořím často přispělo taky k tomu, že jsem tuto konkrétní chybu nehledal.

Za mě rozhodně +.
Můj problém to sice neřeší, jelikož by to bralo, že mám vše ok, jelikož by to bralo jako navěšenou událost z presenteru starající se o přesměrování, ale i tak si myslím, že to je dobrý nápad.

Michal Kumžák napsal(a):

Problém byl, že jsem navěštil tu událost z Presenteru, ale zapomněl jsem navěštit zpracování dat, takže redirect s flashMessage proběhl v pohodě, ale data se nezpracovala. Divil jsem se a docela dlouho na můj vkus jsem hledal kde je chyba a proč se mi nevolá update nad databází.

To by ti v tomto případě, ale tato úprava nepomohla, jestli sem to pochopil správně.

Nepomohla. Proto jsem taky psal:

Můj problém to sice neřeší, …

Jen jsem chtěl poukázat na to, že to je dobrý nápad, jelikož když se to může stát někomu, kdo s Nette pracuje 7 let, tak se to určitě stává nováčkům.

Editoval Polki (30. 8. 2021 16:37)

U GET formulářů by se to asi kontrolovat nemuselo. Proč tam handler cpe parametr?

nazdar, už dlší čas pozorujem tento warning na jednom webe, kde handlery sú zavesené, bolo to ale len sporadicky, no dnes som po tom začal pátrať a zistil som, že mi tento warning generuje asi iba jeden návštevník (jeden typ návštevníka), ktorý má User-agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36 – môže to s tým nejako súvisieť? v priebehu 3 dní to bolo z 50 prihlásení 8×, z toho 2× z jednej IP, 6× z druhej IP, pričom obe IP sú od rovnakého providera, takže považujem za možné sa domnievať, že ide o jednu osobu…

EDIT: project mám cez git nasadený na 10+ weboch, avšak robí mi to iba na jednom

Editoval MajklNajt (20. 1. 2023 8:02)

Podľa logov webservera však tu ide o bežného človeka, ktorý klikne na prihlásenie a formulár odošle. Udalosť vešiam priamo v komponente, nie je však navesená na Form::onSuccess ale SubmitButton::onClick, no to sa tiež kontroluje, tak nechápem :/

EDIT:

92.249.*.* - - [20/Jan/2023:07:03:43 +0100] "GET /catalog HTTP/2.0" 200 7134 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36"
92.249.*.* - - [20/Jan/2023:07:03:43 +0100] "GET /favicon.ico HTTP/2.0" 404 247 "https://***.hu/catalog" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36"
92.249.*.* - - [20/Jan/2023:07:03:46 +0100] "GET /sign/in HTTP/2.0" 200 4361 "https://***.hu/catalog" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36"
92.249.*.* - - [20/Jan/2023:07:03:47 +0100] "POST /sign/in HTTP/2.0" 200 4379 "https://***.hu/sign/in" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36"
92.249.*.* - - [20/Jan/2023:07:03:49 +0100] "POST /sign/in HTTP/2.0" 200 316 "https://***.hu/sign/in" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36"
92.249.*.* - - [20/Jan/2023:07:03:49 +0100] "GET /sign?_fid=uypu HTTP/2.0" 200 5585 "https://***.hu/sign/in" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36"
92.249.*.* - - [20/Jan/2023:07:04:00 +0100] "GET /catalog HTTP/2.0" 200 7441 "https://***.hu/sign?_fid=uypu" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36"
92.249.*.* - - [20/Jan/2023:07:04:35 +0100] "GET /catalog HTTP/2.0" 200 7444 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/109.0.0.0 Safari/537.36"

Editoval MajklNajt (20. 1. 2023 10:54)

Taky bych to potřeboval vypnout. Při použití metody Formulář pro vytvoření i editaci záznamu tak to právě nefunguje.

@Ozzrel

Taky bych to potřeboval vypnout.

Já mám občas formuláře, kde tlačítkem přidávám do formuláře dynamicky nějaký prvek. Toto tlačítko má vypnutou validaci a nemá handler, protože jeho úkolem není provést na serveru nějakou akci, ale pouze změnit stav formuláře (např. počet položek faktury). Abych potlačil warning o chybějícím handleru, musím přidat do formuláře zbytečný handler, který nic nedělá, jenom odstraní ten warning. Čistší řešení by bylo mít možnost tento warning v konfiguraci vypnout.

@mskocik

A nestačilo by ti len nastaviť pre ten button type=button? Ten nesubmituje formulár

V mém případě to není vhodné řešení. Potřebuji, aby formFactory měla hned na začátku když formulář sestavuje informace jak má formulář vypadat. Přesněji, kolik položek faktury má vykreslit. Formulář se sestavuje na serveru, proto když chci, aby klik na tlačítko přidal prvek do formuláře, musím to prohnat POST přes server, aby se současně neztratily data ve formuláři.

Kdybych přidal prvek do formuláře v prohlížeči javascriptem pomocí type=button, tak po odeslání dat nebude při zpracování na serveru formFactory vědět, že tam nějaký prvek/ky přibyl. Musely by se analyzovat odeslaná data POST a to vede na komplikované a nepřehledné řešení. Proto preferuji submit button, formulář si udržuje v pomocném input hidden stav a ten řídí sestavování formuláře ve form Factory.

Při přidávání nových feature do frameworku, je potřeba dát pozor, aby dobře míněná funkce nezpůsobila nezamýšlené potíže. Protože lidi framework používají různými způsoby. V případě upozornění na chybějící handler by bylo vhodné mít možnost tuto hlášku vypnout.

Ahoj, také používám formuláře podobným způsobem jako m.brecher. Řešení je tedy přidat všude prázdný onSuccess? Jiné potlačení warningů není možné?

Mě ne. Bohužel mám v aplikaci stovky různých formulářů, tak jsem chtěl zkusit, jestli někoho zkušenějšího nenapadlo jednodušší řešení…

@elnathan

Všude používám ->onClick[], což je i dle nejnovější dokumentace stále povoleno a v pořádku.

Ve formulářích používám $form->onSuccess[] málokdy. Problém s warningem při chybějícím ovladači onSuccess[] jsem vyřešil ve vlastních abstraktních předcích FormControl a BaseForm – dvojice ze kterých dědí všechny formuláře. Tyto třídy řeší i jiné nepružnosti existující třídy Form a warning řeší nějak takhle:

if($form->onSuccess === []){
    $form->onSuccess[] = fn() => null;        // fix Nette warning
}

Tohle chování by mělo jít globálně vypnout někde v konfiguraci projektu.

Určitě by to bylo vhodné, protože warning jeden problém řeší a jiný vytváří a je otázka zda udělá více škody více než užitku? Přijít na to, že někde chybí ovladač formuláře je práce na minutu, ale přepsat po upgrade Nette všechny formuláře ve všech aplikacích může být práce na mnoho hodin.

Editoval m.brecher (5. 4. 10:31)

Martk napsal(a):

Vypnout warningy jde snadno, ve FormFactory dáš toto:

$form = new Form();
$form->onSuccess[] = fn() => null;
return $form;

při nejhorším ve vlastní třídě Form. A když tvoje aplikace nepoužívá ani jedno, tak si vytvoříš php script, který automaticky vytvoří nové App\Forms\Form a nahradí v php souborech use Nette\Application\UI\Form; za use App\Forms\Form;.

Tady ale nejde o hledání toho, jak to obejít. Proč, když mám vše správně, bych měl dělat nějakou FormFactory, kterou ale k ničemu nepotřebuji? Proč bych měl nahrazovat výchozí Nette Form nějakou svojí Form třídou, když ji nepotřebuji? Jde o to, že v kódu není žádná chyba a jen „kvůli začátečníkům“ to v určitých případech (popsáno v předchozím komentáři) hlásí úplně zbytečně warning. Obcházet to není správný postup (což se přesně váš FormFactory), přepisovat validní formuláře zbytečným kódem taky ne. Tato validace by prostě měla jít vypnout v konfiguraci (neon) nebo tam nebýt vůbec, jelikož bohužel nefuguje dobře s roboty.