WP_LOCAL_DEV shouldn't be included in Site Health Info

[johnbillion]

On the Tools -> Site Health -> Info screen, the WP_LOCAL_DEV constant is included under the heading "WordPress Constants".

This is not a WordPress constant and isn't used by WordPress. Some popular libraries use this constant but it's not a core constant and is misleading to include it under this heading. A colleague had this constant set in the expectation that it affected the return value of wp_get_environment_type(), which it does not.

[Clorith]

Good catch, this seems to be an artifact of the Health Check plugin, where it did make sense to look for it, but the core implementation should definitely only be checking core-related features.

Adding it to the milestone for the next release.

[igorradovanov]

I'd like to work on this one.

[sabbirshouvo]

I want mention this ticket #33161

[bgoewert]

What if we changed WP_LOCAL_DEV to WP_ENVIRONMENT_TYPE since that is the constant used in wp_get_environment_type()?

Even though this value is already displayed under the "WordPress" section on the Site Health Info screen. Does it not make sense to list it's constant under this section as well?

[Clorith]

@bgoewert That sounds like a reasonable solution, yes. Since WP_ENVIRONMENT_TYPE is a core constant, it could safely be displayed, and since the fallback environment, if an invalid value is defined, this is a perfect and quick way to check that value.

[sabbirshouvo]

@Clorith I've added this patch for WP_LOCAL_DEV to WP_ENVIRONMENT_TYPE migration.

[Clorith]

Thank you for the patch @sabbirshouvo.

Would you have time to make a small udpate here, as the output should be the actual value of the constant, and not true, false, Enabled, or Disabled (the Undefined message is OK to leave in though, if the value is not defined).

There's also a few too many spaces on line 321, the number of indents should match the other elements in the array (this isn't a major issue, and we can fix it before committing, but it's good to keep track of :) ), see the line that triggered that warning below (it only needs a single space to align with everything else in the section):

'WP_ENVIRONMENT_TYPE'        => array(

[sabbirshouvo]

@Clorith in addition to your comment, I think $wp_environment_type_debug is also unnecessary here.

[sabbirshouvo]

Adjusted spacing and improved the values

[Clorith]

In 52021:

Site Health: Add the constant WP_ENVIRONMENT_TYPE for debug data.

As a legacy from the old Health Check plugin, the constant WP_LOCAL_DEV was included in the debug data under the "WordPress Constants" section. This was incorrect, and has never been a constant used by core.

Instead, that field is now replaced by WP_ENVIRONMENT_TYPE, which was introduced with WordPress 5.5, and is a much more appropriate value to provide in a debug scenario.

Props johnbillion, bgoewert, sabbirshouvo.
Fixes #54340.