Site Health: Correct markup in ::get_test_php_extensions() and ::get_test_background_updates() description.

SergeyBiryukov · SergeyBiryukov · commit a73eca1021f1 · 2020-03-29T03:44:36.000Z
These tests output an unordered list, which doesn't need to be wrapped in a paragraph tag. Additionally, pass an empty string as an `actions` parameter in `::get_test_php_default_timezone()`, for consistency with other tests. Props afercia. See #48948. git-svn-id: https://develop.svn.wordpress.org/trunk@47528 602fd350-edb4-49c9-b593-d223f7449a82
diff --git a/src/wp-admin/includes/class-wp-site-health.php b/src/wp-admin/includes/class-wp-site-health.php
@@ -1036,10 +1036,7 @@ public function get_test_php_extensions() {
 				$result['label'] = __( 'One or more required modules are missing' );
 			}
 
-			$result['description'] .= sprintf(
-				'<p>%s</p>',
-				$output
-			);
+			$result['description'] .= $output;
 		}
 
 		return $result;
@@ -1064,6 +1061,7 @@ public function get_test_php_default_timezone() {
 				'<p>%s</p>',
 				__( 'PHP default timezone was configured by WordPress on loading. This is necessary for correct calculations of dates and times.' )
 			),
+			'actions'     => '',
 			'test'        => 'php_default_timezone',
 		);
 
@@ -1678,10 +1676,7 @@ public function get_test_background_updates() {
 		$output .= '</ul>';
 
 		if ( 'good' !== $result['status'] ) {
-			$result['description'] .= sprintf(
-				'<p>%s</p>',
-				$output
-			);
+			$result['description'] .= $output;
 		}
 
 		return $result;
