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

Timestamp:

03/29/2020 03:44:36 AM (5 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;
         }