lms icon indicating copy to clipboard operation
lms copied to clipboard

Published 20 hours ago • verified publisher icon chilek

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

Open interduo opened this issue 2 years ago • 3 comments

interduo avatar Sep 22 '22 13:09 interduo

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

chilek avatar Sep 22 '22 13:09 chilek

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

chilek avatar Sep 22 '22 13:09 chilek

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 avatar Sep 22 '22 17:09 interduo

naprawione w 9619792bebada6f26cbd05f57e5784e7f0003621

interduo avatar Dec 30 '23 17:12 interduo