-
Notifications
You must be signed in to change notification settings - Fork 284
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
Review PHP deprecation notices with PHP 8.1+ #8589
Comments
last time there is only one that error: |
there is NO 'compareStartDate' and 'compareEndDate' in keys :
|
I do have a similar error that only appears when Site Kit is active, and the error doesn't reference any Site Kit files: This is on a test site and it occurs with only SK active.
In order to create this error, I had to enable Dashboard Sharing with an editor profile. The error only occurs for this editor. The error also exists with only SC active. Happy to share a login for test purposes. SH info
In my case also, this could be related to my server setup, given I see the below at present for my admin user. Rather than being server related, it's IPv6 related as when I enable the IPv6 plugin, the dashboard loads once more. |
I do have a site active that is portraying similar errors. Error reporting is enabled with PHP 8.1.27 active. These errors consistently appear when connecting any module or when setting up SK.
SH info
|
ok, now it moved v1.131.0 |
IB ✔️ |
QA Update ✅
|
@mohitwp the QAB mentions error logs, so I was wondering if you looked at the log files to ensure no errors were not appearing there too? I don't think TasteWP offers access to these files so you will likely need to set up a test site with our paid InstaWP site to double check. I have moved this back to QA and assigned to you. |
@wpdarren During my test, I enabled error logging on the TasteWP site using the File Manager plugin. I also used the Debug Bar and Query Monitor plugins. However, no errors appeared for me. |
QA Update
|
@mohitwp Thanks. Yes, that will require some digging in a new follow-up issue, since it seems to be coming from 3rd party library. I haven't seen that notice during my testing |
QA Update ✅Thank you, @zutigrm! I will create a separate ticket for you to investigate the PHP warning I mentioned earlier. I’m not seeing any warnings on the TasteWP site; they only appear on the InstaWP site.
|
Bug Description
As reported by one user in the forums, there are deprecation notices stemming from the plugin, and not part of any third party dependency. These can be seem below:
Testing underway, with more insights and log files to be provided.
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Implementation Brief
convertDeprecationsToExceptions
attributeTest Coverage
QA Brief
Changelog entry
The text was updated successfully, but these errors were encountered: