-
Notifications
You must be signed in to change notification settings - Fork 83
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
PHP 8.2 Compatibility #528
Comments
It's a collision with rollbar plugin. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Hello ! edit : I don't have the "rollbar plugin" on my side. |
So we fixed this by forcing certain composer dependencies namely psr/log to use different versions that are compatible with the cloudflare plugin's requirements. IF you are using composer for your project try: If that doesn't fix it then you'll need to keep digging: There is already a PR that would fix this, not sure why they haven't done it yet. #541 |
Confirmation
WordPress version
6.4.2
Cloudflare-WordPress version
4.12.2
PHP version
8.2.8
Expected result
Successful plugin activation
Actual result
$ wp plugin activate cloudflare
PHP Fatal error: Declaration of CF\Integration\DefaultLogger::debug($message, array $context = []) must be compatible with Psr\Log\AbstractLogger::debug(Stringable|string $message, array $context = []): void in /x/wp-content/plugins/cloudflare/src/Integration/DefaultLogger.php on line 42
Fatal error: Declaration of CF\Integration\DefaultLogger::debug($message, array $context = []) must be compatible with Psr\Log\AbstractLogger::debug(Stringable|string $message, array $context = []): void in /x/wp-content/plugins/cloudflare/src/Integration/DefaultLogger.php on line 42
Steps to reproduce
Trying to use plugin with php 8.2.8.
Additional factoids
No response
References
No response
The text was updated successfully, but these errors were encountered: