lms
lms copied to clipboard
chilek
ostrzeżenia w eventadd.php
Bieżąca wersja 28-git z master.
Mamy w logu php:
PHP Warning: Undefined variable $search in /var/www/html/lms/modules/eventadd.php on line 55
Pojawiło się po ostatnich zmianach związanych z przypisaniem węzłów i urządzeń do zdarzeń terminarza. Rzuciłem okiem na skrypt php - wygląda, że użycie $search nie przystaje do jego użycia. Czyżby wynikało to ze zmiany kolejności wierszy kodu?
To, co przygotowałem to naprawia jakiś jeden specyficzny przebieg kodu, ale nie rozwiązuję pierwotnego problemu po zmianie koolejności wierszy w kodzie. Hint: szuka frazy $search - zobaczysz, że występuje w przypadkowej kolejności, a to znaczy, że trafia do wywołania korzystającego z tego nie wszystko, co trzeba przekazując.
Właśnie przez jakiś taki PHP Warning dodałem test !empty() wewnątrz metody GetNetDevList().
BTW używasz na produkcji już PHP8.0 czy to jakaś testowa instancja o której piszesz? Dogrywam właśnie PHP8.x na testowej instancji zaraz to wrzucę na debugger.
Przeniosłem to wyżej, żeby nie wykonywać dwa razy tej samej metody - wyczyściłem ten warunek ponieważ nie jest on potrzebny i przy okazji znalazłem kilka drobnych usprawnień, które można wprowadzić w kod.
naprawione w https://github.com/chilek/lms/commit/9619792bebada6f26cbd05f57e5784e7f0003621