Skip to content

Conversation

@ocean90
Copy link
Member

@ocean90 ocean90 commented Jan 22, 2022

Strip off any -alpha, -RC, -beta, -src suffixes before using version_compare(). Also, replace get_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


This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

Copy link
Contributor

@costdev costdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should unit tests be added to cover this change?

If not, LGTM 👍

@dd32
Copy link
Member

dd32 commented Jan 24, 2022

LGTM too, I don't think unit tests are really needed for this specific change.

@ocean90 ocean90 closed this Jan 24, 2022
@ocean90 ocean90 deleted the fix/wp-compatible-check branch January 24, 2022 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants