#54882 closed defect (bug) (fixed)
Allow to install/activate themes/plugin which require the next WordPress version
Reported by: | ocean90 | Owned by: | ocean90 |
---|---|---|---|
Milestone: | 5.9 | Priority: | normal |
Severity: | normal | Version: | 5.9 |
Component: | Plugins | Keywords: | has-patch has-unit-tests dev-reviewed commit |
Focuses: | administration | Cc: |
Description
I have noticed this while creating a plugin to disable the login language switcher. As it's a feature introduced in 5.9 I have set the minimum required WP version to 5.9. Unfortunately with that setup it's not possible to test the plugin with any a beta or RC release, see attached screenshot.
The same came up after the version was bumped for Twenty Twenty-Two in [52614]. The "fix" was to skip the tests in [52617].
Im proposing to change is_wp_version_compatible()
to remove any custom suffixes to ensure version_compare()
can do its job right.
While is_wp_version_compatible()
was introduced in 5.2, the use of the function in the list tables was added in [51678].
Note: If you think "why not set the required version to 5.9-alpha", this isn't supported, see [meta7874].
Set the milestone to 5.9.1 since it's probably too late for 5.9 although it would allow us to revert [52617].
Attachments (3)
Change History (20)
This ticket was mentioned in PR #2211 on WordPress/wordpress-develop by ocean90.
3 years ago
#1
- Keywords has-patch added
This ticket was mentioned in Slack in #core by ocean90. View the logs.
3 years ago
This ticket was mentioned in PR #2212 on WordPress/wordpress-develop by ocean90.
3 years ago
#3
- Keywords has-unit-tests added
https://github.com/WordPress/wordpress-develop/pull/2211 for the 5.9 branch to test the revert.
Trac ticket: https://core.trac.wordpress.org/ticket/54882
This ticket was mentioned in Slack in #core-themes by costdev. View the logs.
3 years ago
#7
@
3 years ago
When I was testing with 5.9-RC3, I was also troubled because once I deactivated TT2, I couldn't reactivate it. The patches work fine for me.
This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.
3 years ago
#11
@
3 years ago
- Owner set to ocean90
- Resolution set to fixed
- Status changed from new to closed
In 52628:
#12
@
3 years ago
- Keywords dev-feedback added
- Milestone changed from 5.9.1 to 5.9
- Resolution fixed deleted
- Status changed from closed to reopened
It appears a RC4 is planned so I'm proposing to revert [52617] and backport [52628] to 5.9. See https://github.com/WordPress/wordpress-develop/pull/2212 for passing 5.9 tests.
Strip off any -alpha, -RC, -beta, -src suffixes before using
version_compare()
. Also, replaceget_bloginfo()
call with the global to ensure that plugins filtering the version for "security purposes" don't let the compatibility check fail.Trac ticket: https://core.trac.wordpress.org/ticket/54882