-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
Warnings on PHP 8.x #680
Comments
Hello! TSF is compatible with PHP 7.4 ~ 8.3. The other ticket is about increasing the minimum requirements. Our websites currently work with PHP 8.1. The plugin shouldn't emit warnings on any of the supported versions. But when it does, it's highly likely something on the site isn't working according to the standard APIs. Here are some scenarios I figured from a dry run (ranked from difficulty to verify). With these scenarios, I expect you can see the warnings on your screen and not only via logs:
Did any of this help find the cause? |
From other reports I've assimilated that there is a plugin active that calls the meta box before WordPress establishes what a post or page is. This meta box, with the SEO Bar embedded, tries to call a non-existing post object's metadata. This shouldn't be a cause for concern, but it is a bug in another plugin, and it does indeed spawn unwanted errors. These errors will no longer spawn from TSF v5.0.7 because we'll then always return the defaults: 5bf612f. It doesn't resolve the issue where the meta box is called for no good reason, however. |
We have plans to upgrade to 8.2 here #644
In the meantime would it be good to fix some warnings on php8.x?
The text was updated successfully, but these errors were encountered: