Starting with PHP 8.1 the JsonSerializable interface has a return type of mixed for its jsonSerialize method which is implemented in Minishlink\WebPush\MessageSentReport.
Adding a return type of mixed will break the library in all versions before PHP 8.1. Not making any changes will break the library in PHP 8.1.
However, adding the attribute #[\ReturnTypeWillChange] allows us to make no further changes but ensure the library will still work in all existing versions AND PHP 8.1.
The attribute is entirely ignored in non-supporting PHP versions.
So, as per the contributing guidelines this has not been tested but you can see for yourself it does not in the slightest change the functionality so should be good to merge.
Note: There needs to be similar changes in the JWT libraries but I'll be doing some PRs for those too shortly.
Starting with PHP 8.1 the
JsonSerializable
interface has a return type ofmixed
for itsjsonSerialize
method which is implemented inMinishlink\WebPush\MessageSentReport
.Adding a return type of
mixed
will break the library in all versions before PHP 8.1. Not making any changes will break the library in PHP 8.1.However, adding the attribute
#[\ReturnTypeWillChange]
allows us to make no further changes but ensure the library will still work in all existing versions AND PHP 8.1.The attribute is entirely ignored in non-supporting PHP versions.
So, as per the contributing guidelines this has not been tested but you can see for yourself it does not in the slightest change the functionality so should be good to merge.
Note: There needs to be similar changes in the JWT libraries but I'll be doing some PRs for those too shortly.
Cheers!