Neexistujuci index ceske_sluzby_dodaci_doba v $_POST

pavelevap / ceske-sluzby

České služby pro WordPress

56 stars 25 forks source link

Neexistujuci index ceske_sluzby_dodaci_doba v $_POST #157

Closed marian-kadanka closed 7 years ago

marian-kadanka commented 7 years ago

Pri vytvarani a uprave produktu sa v logu objavuje PHP Notice: Undefined index: ceske_sluzby_dodaci_doba - pridame kontrolu

pavelevap commented 7 years ago

Díky moc, tohle mi nějak uniklo :-(

Nebylo by lepší (v případě že $_POST['ceske_sluzby_dodaci_doba'] vůbec nebude definováno), celý další ukládací skript pro dodací dobu zcela přeskočit? Alespoň by se ani nespouštěly zbytečně funkce get_post_meta() a další...

A koukám, že u variant mi chybí kontrola také pro $_POST['ceske_sluzby_hodnota_ean'][$i] :-(

marian-kadanka commented 7 years ago

Ano, upravil som to, aby sa uplne ukladanie preskocilo, ak prislusne hodnoty nie su v $_POST definovane.

Pridal som aj kontrolu pre 'ceske_sluzby_hodnota_ean'.

pavelevap commented 7 years ago

Ještě jednou: díky moc!

Přemýšlím, zda by neměla být kontrola také na $_POST['ceske_sluzby_dodaci_doba'][$i] a ne jen na celou hodnotu $_POST['ceske_sluzby_dodaci_doba'], ale asi to bude jedno, protože když tam bude alespoň něco, tak to musí být vlastně všude a nelze se pak snad dostat k problému, že pro konkrétní $i není $_POST['ceske_sluzby_dodaci_doba'][$i] definováno?