Changeset 56570 for trunk/src/wp-admin/site-health.php

Timestamp:

09/14/2023 12:52:45 AM (15 months ago)

Author:

joedolson

Message:

Administration: Use wp_admin_notice() in /wp-admin/.

Add usages of wp_admin_notice() and wp_get_admin_notice() on .notice-[type] in the root level of /wp-admin/. Ongoing task to implement new function across core.

Props costdev, joedolson.
See #57791.

File:

• trunk/src/wp-admin/site-health.php (modified) (2 diffs)

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

@@ -106 +106 @@
     if ( isset( $_GET['https_updated'] ) ) {
         if ( $_GET['https_updated'] ) {
-            ?>
-            <div id="message" class="notice notice-success is-dismissible"><p><?php _e( 'Site URLs switched to HTTPS.' ); ?></p></div>
-            <?php
+            wp_admin_notice(
+                __( 'Site URLs switched to HTTPS.' ),
+                array(
+                    'type'        => 'success',
+                    'id'          => 'message',
+                    'dismissible' => true,
+                )
+            );
         } else {
-            ?>
-            <div id="message" class="notice notice-error is-dismissible"><p><?php _e( 'Site URLs could not be switched to HTTPS.' ); ?></p></div>
-            <?php
+            wp_admin_notice(
+                __( 'Site URLs could not be switched to HTTPS.' ),
+                array(
+                    'type'        => 'error',
+                    'id'          => 'message',
+                    'dismissible' => true,
+                )
+            );
         }
     }
@@ -213 +223 @@
     return;
 } else {
+    wp_admin_notice(
+        __( 'The Site Health check requires JavaScript.' ),
+        array(
+            'type'               => 'error',
+            'additional_classes' => array( 'hide-if-js' ),
+        )
+    );
     ?>
-
-<div class="notice notice-error hide-if-js">
-    <p><?php _e( 'The Site Health check requires JavaScript.' ); ?></p>
-</div>
 
 <div class="health-check-body health-check-status-tab hide-if-no-js">