search

dbojdo / wFirma

wFirma SDK
MIT License
13 stars 15 forks source link

Fix PHP 8.1 compatibility #42

Open mgralikowski opened 1 year ago

mgralikowski commented 1 year ago

Po aktualizacji do PHP 8.1 w logach pojawiły się tego typu warningi:

Return type of Webit\WFirmaSDK\Entity\EntityIterator::rewind() should either be compatible with Iterator::rewind(): void, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /vendor/webit/w-firma-api/src/Entity/EntityIterator.php on line 78

Ten PR powinien je usunąć.

jacekkow commented 1 year ago

mixed jako typ zwracany zostało dodane dopiero w PHP 8.0.

Master ma zachowywać kompatybilność z 7.1 (patrz https://github.com/dbojdo/wFirma/blob/master/composer.json#L8), chyba że właściciel repo pozwoli podbić wersję i wymagania ;)

mgralikowski commented 1 year ago

@dbojdo to proponuje utworzyć nowy branch 3.x, od którego minimum będzie php 8.0.

dbojdo commented 1 year ago

Tak, mozemy zrobic branch 3.0, zrobic port do PHP8.0. Jakiś ochotnik?