Opened 6 years ago
Closed 5 years ago
#47299 closed enhancement (fixed)
Site Health: get_test_theme_version() + WP_DEFAULT_THEME = use get_core_default_theme() !
Reported by: | juliobox | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.3 | Priority: | normal |
Severity: | minor | Version: | 5.2 |
Component: | Site Health | Keywords: | 2nd-opinion site-health has-patch |
Focuses: | Cc: |
Description
Hello there
The test for get_test_theme_version() is checking if WP_DEFAULT_THEME is set to one of our installed theme. If not, the SH will tell us to use one.
But since I have a twentystuff theme on my installation, WordPress will find a fallback. Thanks to get_core_default_theme()!
We should also use this as a check.
I just did this:
<?php if ( ! $has_default_theme ) {
becomes:
<?php if ( ! $has_default_theme && ! WP_Theme::get_core_default_theme() ) {
That does the correct test I guess.
Thank you!
Attachments (1)
Change History (7)
Note: See
TracTickets for help on using
tickets.
Makes sense to me,
populate_options()
andvalidate_current_theme()
do the same.