Site Health: debug info prints out list items with no wrapping list

[afercia]

In the "Info" page, within the accordions, some debug info are printed out as list items.

However, these list items miss a wrapping <ul> element thus producing invalid markup. Also, the list semantics is absent and assistive technologies can't announce this markup as a list.

See https://core.trac.wordpress.org/browser/trunk/src/wp-admin/site-health-info.php?rev=45044&marks=140-146#L137

if ( is_array( $field['value'] ) ) {
	$values = '';
	foreach ( $field['value'] as $name => $value ) {
		$values .= sprintf(
			'<li>%s: %s</li>',
			esc_html( $name ),
			esc_html( $value )
		);
	}
}

[mukesh27]

Patch that wrap ul tag

[chetan200891]

@mukesh27 Thanks for patch. Can you add one space for first $values to fix PHPCS issue. Also I think we can now directly assign <ul> to $values. No need blank $values.

[mukesh27]

Updated Patch as suggested by @chetan200891

[SergeyBiryukov]

In 45068:

Site Health: Add missing <ul> tags for lists on Site Info page.

Props mukesh27, chetan200891, afercia.
Fixes #46693.