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,
+                                )
+                        );
                 }