-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecated functions on PHP 8.1 #88
Comments
In addition to all the iterator functions of the
Is there any progress to expect on the PHP 8.1 compatibility? |
@jchoca Sorry to jump in to this, but could you have a look at this, or ping the correct people at Wildbit? |
Hello @malles! I am attempting to help get this library up to date for PHP 8.x. I have a PR for suppressing these reported deprecation warnings (#95). Can you try it out with your use case(s) and let me know if you see any other functions that need suppression still? Since we still support 7.x we need to suppress these for now, as older PHP versions do not support using the |
Thank you for picking this up! I do get some more warnings about return types in
That is what would fix it for my code path, but I'm not sure if that is sufficient for all users. Pullrequest #90 for example is about warnings in |
Thanks again. I think this will stop most immediate bleeding. I do think it would be wise to consider #94 to further future proof this package. |
#95 has been merged. |
I see the issue was closed months ago however I'm seeing these errors in the Postmark Drupal module version 8.x-1.1 , Drupal 9.4.8 , PHP 8.1.11 on Apache 2.4.54
|
Drupal 9.5.1, PHP 8.1
|
And... we gotta go to the postmark module developers.... They're still requiring ^2.5 |
Deprecated function: Return type of
Postmark\Models\CaseInsensitiveArray::offsetExists($offset)
should either be compatible with ArrayAccess::offsetExists(mixed $offset):
bool, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress
the notice in include()
Deprecated function:
Return type of Postmark\Models\CaseInsensitiveArray::offsetGet($offset)
should either be compatible with ArrayAccess::offsetGet(mixed $offset): mixed,
or the #[\ReturnTypeWillChange] attribute
should be used to temporarily suppress
the notice in include()
started Getting Notices when upgraded to 8.1
The text was updated successfully, but these errors were encountered: