-
Notifications
You must be signed in to change notification settings - Fork 397
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
jsonSerialize missing return type messages in PHP 8.1 #429
Comments
These deprecation notices were annoying enough that I forked and patched the code on our servers with the I can't submit a pull request for this, apparently, as per the project guidelines:
Temporary fixes:
|
@LordPachelbel thank you for fork. PR's seem to be stuck for PHP 8.1 issues. Anet team, please get these PHP 8.1 updates pulled in. |
@Zelfapp To be clear, my fork only suppresses the messages. The AuthorizeNet team might want to instead change the return type to |
PHP 8.1 outputs several deprecation messages because the
jsonSerialize()
methods implemented by this package do not specify themixed
return type nor use the#[\ReturnTypeWillChange]
attribute.The message doesn't appear in PHP 8.0.
Sample message:
Affected files:
lib/net/authorize/api/contract/v1/*.php
Suggested fix:
Suppress the notice by adding the
#[\ReturnTypeWillChange]
attribute above the method's implementation since themixed
return type is only available as of PHP 8.0.The text was updated successfully, but these errors were encountered: