Changeset 56599 for trunk/src/wp-admin/includes/dashboard.php

Timestamp:

09/17/2023 03:21:07 PM (20 months ago)

Author:

costdev

Message:

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

Adds further usages of wp_admin_notice() in /wp-admin/includes/ on .notice-error, .notice-warning, .error, and .updated.

Ongoing task to implement new function across core.

Follow-up to [56408], [56409], [56410], [56518], [56570], [56571], [56572], [56573], [56576], [56589], [56590], [56597].

Props joedolson, mukesh27, costdev.
See #57791.

File:

• trunk/src/wp-admin/includes/dashboard.php (modified) (3 diffs)

trunk/src/wp-admin/includes/dashboard.php

@@ -574 +574 @@
     <form name="post" action="<?php echo esc_url( admin_url( 'post.php' ) ); ?>" method="post" id="quick-press" class="initial-form hide-if-no-js">
 
-        <?php if ( $error_msg ) : ?>
-        <div class="error"><?php echo $error_msg; ?></div>
-        <?php endif; ?>
+        <?php
+        if ( $error_msg ) {
+            wp_admin_notice(
+                $error_msg,
+                array(
+                    'additional_classes' => array( 'error' ),
+                )
+            );
+        }
+        ?>
 
         <div class="input-text-wrap" id="title-wrap">
@@ -1158 +1165 @@
  */
 function wp_dashboard_cached_rss_widget( $widget_id, $callback, $check_urls = array(), ...$args ) {
-    $loading    = '<p class="widget-loading hide-if-no-js">' . __( 'Loading&hellip;' ) . '</p><div class="hide-if-js notice notice-error inline"><p>' . __( 'This widget requires JavaScript.' ) . '</p></div>';
     $doing_ajax = wp_doing_ajax();
+    $loading    = '<p class="widget-loading hide-if-no-js">' . __( 'Loading&hellip;' ) . '</p>';
+    $loading   .= wp_get_admin_notice(
+        __( 'This widget requires JavaScript.' ),
+        array(
+            'type'               => 'error',
+            'additional_classes' => array( 'inline', 'hide-if-js' ),
+        )
+    );
 
     if ( empty( $check_urls ) ) {
@@ -1341 +1355 @@
  */
 function wp_print_community_events_markup() {
-    ?>
-
-    <div class="community-events-errors notice notice-error inline hide-if-js">
-        <p class="hide-if-js">
-            <?php _e( 'This widget requires JavaScript.' ); ?>
-        </p>
-
-        <p class="community-events-error-occurred" aria-hidden="true">
-            <?php _e( 'An error occurred. Please try again.' ); ?>
-        </p>
-
-        <p class="community-events-could-not-locate" aria-hidden="true"></p>
-    </div>
-
-    <div class="community-events-loading hide-if-no-js">
-        <?php _e( 'Loading&hellip;' ); ?>
-    </div>
-
-    <?php
+    $community_events_notice  = '<p class="hide-if-js">' . ( 'This widget requires JavaScript.' ) . '</p>';
+    $community_events_notice .= '<p class="community-events-error-occurred" aria-hidden="true">' . __( 'An error occurred. Please try again.' ) . '</p>';
+    $community_events_notice .= '<p class="community-events-could-not-locate" aria-hidden="true"></p>';
+
+    wp_admin_notice(
+        $community_events_notice,
+        array(
+            'type'               => 'error',
+            'additional_classes' => array( 'community-events-errors', 'inline', 'hide-if-js' ),
+            'paragraph_wrap'     => false,
+        )
+    );
+
     /*
      * Hide the main element when the page first loads, because the content