Site Health: Display both site and user locale in debug information

[desrosj]

In some scenarios, it could be useful to know when the site's locale differs from the user's preference.

In the debug info, only one "Language" field is provided, indicating the current user's locale, either from selected preference or site default.

I propose changing this field to "User Language" and adding "Site Language". This could prove useful for debugging i18n issues.

Follow up of #46694.

[mukesh27]

@desrosj, Patch looks good to me. one small suggestion, As we remove "language" from array list can we remove $original_locale = get_user_locale(); from debug_data function as $original_locale is not used any where in that function now.

[ianbelanger]

Updates patch to remove suggested variable

[ianbelanger]

Thanks for the suggestion @mukesh27, I have updated the original patch to remove $original_locale = get_user_locale(); as it is no longer required. There do not seem to be any adverse affects.

[Clorith]

This needs a refresh, as the patch now doesn't apply cleanly.

[mukesh27]

Updated patch.

[Clorith]

Looks good.

[desrosj]

In 45166:

Site Health: Include both site and user locales in debug information.

Previously, only the site’s locale was displayed in the Site Health debug information. Moving forward, both the site’s default locale and the user’s locale will be specified.

This change also removes the @param tag for WP_Debug_Data::debug_data(). This parameter was previously removed in [45156].

Props: mukesh27, ianbelanger, Clorith, desrosj.
Fixes #46803.