Changeset 56600 for trunk/src/wp-admin/nav-menus.php

Timestamp:

09/17/2023 03:31:32 PM (3 years ago)

Author:

joedolson

Message:

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

Add additional usage of wp_admin_notice() in wp-admin/ on .error and miscellaneous usages previously overlooked.

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

Props costdev, joedolson.
See #57791.

File:

• trunk/src/wp-admin/nav-menus.php (modified) (4 diffs)

trunk/src/wp-admin/nav-menus.php

@@ -385 +385 @@
 
                 if ( is_wp_error( $_nav_menu_selected_id ) ) {
-                    $messages[] = '<div id="message" class="error notice is-dismissible"><p>' . $_nav_menu_selected_id->get_error_message() . '</p></div>';
+                    $messages[] = wp_get_admin_notice(
+                        $_nav_menu_selected_id->get_error_message(),
+                        array(
+                            'id'                 => 'message',
+                            'additional_classes' => array( 'error' ),
+                            'dismissible'        => true,
+                        )
+                    );
                 } else {
                     $_menu_object            = wp_get_nav_menu_object( $_nav_menu_selected_id );
@@ -436 +443 @@
                 }
             } else {
-                $messages[] = '<div id="message" class="error notice is-dismissible"><p>' . __( 'Please enter a valid menu name.' ) . '</p></div>';
+                $messages[] = wp_get_admin_notice(
+                    __( 'Please enter a valid menu name.' ),
+                    array(
+                        'id'                 => 'message',
+                        'additional_classes' => array( 'error' ),
+                        'dismissible'        => true,
+                    )
+                );
             }
 
@@ -458 +472 @@
 
             if ( ! $menu_title ) {
-                $messages[] = '<div id="message" class="error notice is-dismissible"><p>' . __( 'Please enter a valid menu name.' ) . '</p></div>';
+                $messages[] = wp_get_admin_notice(
+                    __( 'Please enter a valid menu name.' ),
+                    array(
+                        'id'                 => 'message',
+                        'additional_classes' => array( 'error' ),
+                        'dismissible'        => true,
+                    )
+                );
                 $menu_title = $_menu_object->name;
             }
@@ -511 +532 @@
             set_theme_mod( 'nav_menu_locations', $menu_locations );
 
-            $messages[] = '<div id="message" class="updated notice is-dismissible"><p>' . __( 'Menu locations updated.' ) . '</p></div>';
+            $messages[] = wp_get_admin_notice(
+                __( 'Menu locations updated.' ),
+                array(
+                    'id'                 => 'message',
+                    'additional_classes' => array( 'updated' ),
+                    'dismissible'        => true,
+                )
+            );
         }