ostrzeżenia na PHP >= 8.0

[chilek]

Wiersz:

https://github.com/BillTechPL/lms-billtech-plugin/blob/04a982e3cc6b0f150a14e26c6a6afb8e6b53a9d0/handlers/BillTechLinkInsertHandler.php#L186

potrafi generować ostrzeżenie PHP (chyba od PHP 8.0):

[22-Jul-2022 13:25:32 Europe/Warsaw] PHP Warning:  Attempt to read property "link" on null in /var/www/html/lms/plugins/BillTech/handlers/BillTechLinkInsertHandler.php on line 186

Ogólnie do poprawienia proste, ale czy $linksManager->getBalanceLink($customerId, ['utm_medium' => 'userpanel']) faktycznie ma prawo zwracać null?

[chilek]

Również tu:

https://github.com/BillTechPL/lms-billtech-plugin/blob/04a982e3cc6b0f150a14e26c6a6afb8e6b53a9d0/handlers/BillTechLinkInsertHandler.php#L39

generuje:

[22-Jul-2022 13:36:55 Europe/Warsaw] PHP Warning:  Attempt to read property "shortLink" on null in /var/www/html/lms/plugins/BillTech/handlers/BillTechLinkInsertHandler.php on line 39

Wynika z tego, że $linksManager->getBalanceLink($customerId, $params) może zwracać null?

[pablos-billtech]

@chilek Dzięki za zgłoszenie. Wydaje mi się, że są 2 możliwości, gdy $linksManager->getBalanceLink($customerId, $params) zwróci null. Obie mają związek z tym, że w bazie nie będzie wygenerowanych linków do płatności:

• Pierwsza sytuacja jest taka, że linki nie zdążą się jeszcze wygenerować lub użytkownik nie ma jeszcze żadnych wystawionych faktur (a w konsekwencji linków), a użytkownik wejdzie na userpanel.
• Druga opcja to próba wysłania wiadomości przez messageadd do użytkownika, który nie ma jeszcze wystawionych żadnych linków do płatności.

Czyli aby temu zapobiec najlepiej będzie te 2 opcje znaleźć w kodzie (najpewniej tam, gdzie się pojawiają powyższe warningi) i dodać tam sprawdzenie.

[chilek]

Trochę warningów wziętych z COMAX:

[05-Aug-2022 12:46:10 Europe/Warsaw] PHP Warning:  Attempt to read property "link" on null in /var/www/html/lms/plugins/BillTech/handlers/BillTechLinkInsertHandler.php on line 51
[05-Aug-2022 13:24:51 Europe/Warsaw] PHP Warning:  Undefined array key 354221 in /var/www/html/lms/plugins/BillTech/lib/BillTechLinksManager.php on line 372
[05-Aug-2022 13:24:51 Europe/Warsaw] PHP Warning:  Undefined array key 354221 in /var/www/html/lms/plugins/BillTech/lib/BillTechLinksManager.php on line 372
[05-Aug-2022 13:24:51 Europe/Warsaw] PHP Warning:  Undefined property: BillTechLinksManager::$debug in /var/www/html/lms/plugins/BillTech/lib/BillTechLinksManager.php on line 287
[05-Aug-2022 13:24:51 Europe/Warsaw] PHP Warning:  Attempt to read property "link" on null in /var/www/html/lms/plugins/BillTech/handlers/BillTechLinkInsertHandler.php on line 186

[chilek]

Z MyNET:

[05-Aug-2022 14:12:35 Europe/Warsaw] PHP Deprecated:  Required parameter $order follows optional parameter $search in /var/www/html/lms/plugins/BillTech/modules/billtechpaymentlist.php on line 3

[pablos-billtech]

Trochę warningów wziętych z COMAX: [05-Aug-2022 12:46:10 Europe/Warsaw] PHP Warning: Attempt to read property "link" on null in /var/www/html/lms/plugins/BillTech/handlers/BillTechLinkInsertHandler.php on line 51 [05-Aug-2022 13:24:51 Europe/Warsaw] PHP Warning: Undefined array key 354221 in /var/www/html/lms/plugins/BillTech/lib/BillTechLinksManager.php on line 372 [05-Aug-2022 13:24:51 Europe/Warsaw] PHP Warning: Undefined array key 354221 in /var/www/html/lms/plugins/BillTech/lib/BillTechLinksManager.php on line 372 [05-Aug-2022 13:24:51 Europe/Warsaw] PHP Warning: Undefined property: BillTechLinksManager::$debug in /var/www/html/lms/plugins/BillTech/lib/BillTechLinksManager.php on line 287 [05-Aug-2022 13:24:51 Europe/Warsaw] PHP Warning: Attempt to read property "link" on null in /var/www/html/lms/plugins/BillTech/handlers/BillTechLinkInsertHandler.php on line 186

W pull request #93 dodałem zmiany rozwiązujące wszystkie alerty oprócz PHP Warning: Undefined array key 354221 in /var/www/html/lms/plugins/BillTech/lib/BillTechLinksManager.php on line 372, które wejdzie w innym pull request.

[chilek]

@pablos-billtech kiedy czekające PR-y wejdą do master?

[pablos-billtech]

PRy weszły