Changeset 47528 for trunk/src/wp-admin/includes/class-wp-site-health.php

Timestamp:

03/29/2020 03:44:36 AM (6 years ago)

Author:

SergeyBiryukov

Message:

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

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.

File:

• trunk/src/wp-admin/includes/class-wp-site-health.php (modified) (3 diffs)

trunk/src/wp-admin/includes/class-wp-site-health.php

@@ -1037 +1037 @@
                         }
 
-                        $result['description'] .= sprintf(
-                                '<p>%s</p>',
-                                $output
-                        );
+                        $result['description'] .= $output;
                 }
 
@@ -1065 +1062 @@
                                 __( 'PHP default timezone was configured by WordPress on loading. This is necessary for correct calculations of dates and times.' )
                         ),
+                        'actions'     => '',
                         'test'        => 'php_default_timezone',
                 );
@@ -1679 +1677 @@
 
                 if ( 'good' !== $result['status'] ) {
-                        $result['description'] .= sprintf(
-                                '<p>%s</p>',
-                                $output
-                        );
+                        $result['description'] .= $output;
                 }