search

TIS2023-FMFI / pracovne-cesty

Projekt TIS 2023 pre Katedru aplikovanej informatiky
The Unlicense
2 stars 0 forks source link

Pri chybnej url sa resetujú niektoré polia v zozname #136

Open kocurvik opened 7 months ago

kocurvik commented 7 months ago

Dobrý deň,

do pola link na udalosť som dal pôvodne adresu cescg.org. Po pokuse o pridanie cesty som dostal oznam, že url nie je správna, tak som pred ňu pridal https://.

To fungovalo, ale resetovali sa polia:

Omylom som tak poslal zlé údaje okrem ŠPP prvku, ktorý som si všimol. Výber polí by mal aj po chybe ostať tak ako som ho zvolil, alebo by som na to mal byť aspoň upozornený.

Mcibula commented 7 months ago

Dobrý deň, ďakujeme, pozrieme sa na to.

pavelpetrovic commented 7 months ago

Ano, selectom sa pri neuspesnej validacii resetuje vyber na prvy v zozname.

Tu niekde (a v ostatnych x-dropdown-input) asi treba povedat, ze ma pouzit "old value", ale neviem ako to funguje v Blade, nasiel som navody iba pre obycajny select pre Laravel...

https://github.com/TIS2023-FMFI/pracovne-cesty/blob/b3fda0f3861d62def0944684f5d352495269966b/src/resources/views/business-trips/create.blade.php#L77C25-L77C43

mozno ze to pre x-dropdown ani nie je a bude treba pouzit ten select namiesto toho (?) https://stackoverflow.com/questions/78070615/how-to-keep-select-selected-with-the-old-function-laravel-blade

silviabielikova commented 7 months ago

ta funkcionalita s old value je naprogramovana priamo v komponente dropdown-input (https://github.com/TIS2023-FMFI/pracovne-cesty/blob/b3fda0f3861d62def0944684f5d352495269966b/src/resources/views/components/dropdown-input.blade.php#L20), teda sa pouziva pre kazdy. dropdown-input je nas vlastny komponent, ktory som robila, aby ponukal uz vsetku funckonalitu a mal rovnake stylovanie (teda je to wrapper pre select element) problem moze byt ale aj inde, napada mi napriklad validacia formularu (validuje sa po castiach). ale momentalne neviem. mozeme sa na to pozriek, ked bude viac casu, bohuzial teraz vobec nie je.

pavelpetrovic commented 6 months ago

@silviabielikova problem je v tom, ze ten ":selected" je pouzity iba v resources/views/business-trips/edit.blade.php ale pocas zadavania cesty sa udaje nahadzuju cez resources/views/business-trips/create.blade.php kde to pouzite nie je. ked som to tam len tak jednoducho skusal pridat, tak to padlo, lebo tam $trip nic nie je. ani ked sa to schova za isset($trip) to nepomaha, tam to treba nejak inak. nemate nejaky hint kadial by mohla ist cesta?

pavelpetrovic commented 6 months ago

nejako som to teda v tomto jednom formulare navela navela v tu vysie odkazanom komite vyriesil, ale nebude to jediny formular, kde sa to deje. este nieco podobne robi aj adminovi, skontrolujem neskor. neviem preco, to neslo tak, ako to robite, blade je pre mna spanielska dedina...

pavelpetrovic commented 6 months ago

doplnil som este na dalsie miesta nefungujuce dropboxy, commit 7266e4f ani v tom edite to old() nebolo funkcne spravene, nieco je tam cele zle.

rovnakym problemom vsak su aj checkboxy, ktore tiez bohuzial nefunguju. :( takze ak clovek vyklika niektore dni, kedy nechce stravu, ale vo formulari ma nejaku chybu, tak po odoslani formulara o to vyklikanie pride, vymaze sa. treba este opravit aj to...