bugfix: remove php 8.x warning msg (lmsplus+/#2322)

chilek / lms

Lan Management System (LMS) public GIT repo

http://lms.org.pl

126 stars 135 forks source link

bugfix: remove php 8.x warning msg (lmsplus+/#2322) #2323

Closed interduo closed 10 months ago

chilek commented 2 years ago

To tylko jakiś specyficzny przebieg kodu naprawia. Czytaj moje równoległe zgłoszenie, w którym opisuję w czym zasadniczy problem.

chilek commented 2 years ago

Spójrz na to: https://github.com/chilek/lms/blob/0243c87ae0bdf1a689f6f7de73ea0bb65aa1756f/modules/eventadd.php#L487-L491 Przygotowujemy $search, a potem nigdzie nie używamy.

interduo commented 2 years ago

ok - poczyściłem eventadd.php trochę, idzie to tylko do mastera więc nie ma sensu tego jakoś rozdzielać. Oczywiście przy okazji znalazłem jeszcze kilka rzeczy: https://github.com/chilek/lms/pull/2324

Przy przenoszeniu w górę (po to by nie odpalać metody dwa razy) zapomniałem usunąć fragmentu niżej (zostawiłem martwy kod). Poprawiłem.

interduo commented 10 months ago

naprawione w 9619792bebada6f26cbd05f57e5784e7f0003621