Changeset 56572

Timestamp:

09/14/2023 01:23:37 AM (15 months ago)

Author:

joedolson

Message:

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

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

Props costdev, joedolson.
See #57791.

Location:

trunk/src/wp-includes

Files:

• customize/class-wp-customize-theme-control.php (modified) (4 diffs)
• user.php (modified) (1 diff)

trunk/src/wp-includes/customize/class-wp-customize-theme-control.php

@@ -241 +241 @@
                     </div>
                 </div>
-                <div class="notice notice-success notice-alt"><p><?php _ex( 'Installed', 'theme' ); ?></p></div>
+                <?php
+                wp_admin_notice(
+                    _x( 'Installed', 'theme' ),
+                    array(
+                        'type'               => 'success',
+                        'additional_classes' => array( 'notice-alt' ),
+                    )
+                );
+                ?>
             <# } else if ( 'installed' === data.theme.type ) { #>
                 <# if ( data.theme.blockTheme ) { #>
@@ -256 +264 @@
                         </div>
                     </div>
-                    <div class="notice notice-error notice-alt"><p>
-                    <?php
-                        _e( 'This theme doesn\'t support Customizer.' );
-                    ?>
+                    <?php $customizer_not_supported_message = __( 'This theme doesn\'t support Customizer.' ); ?>
                     <# if ( data.theme.actions.activate ) { #>
                         <?php
-                            echo ' ';
-                            printf(
+                            $customizer_not_supported_message .= ' ' . sprintf(
                                 /* translators: %s: URL to the themes page (also it activates the theme). */
                                 __( 'However, you can still <a href="%s">activate this theme</a>, and use the Site Editor to customize it.' ),
@@ -270 +274 @@
                         ?>
                     <# } #>
-                    </p></div>
+
+                    <?php
+                    wp_admin_notice(
+                        $customizer_not_supported_message,
+                        array(
+                            'type'               => 'error',
+                            'additional_classes' => array( 'notice-alt' ),
+                        )
+                    );
+                    ?>
                 <# } else { #>
                     <div class="theme-id-container">
@@ -282 +295 @@
                         </div>
                     </div>
-                    <div class="notice notice-success notice-alt"><p><?php _ex( 'Installed', 'theme' ); ?></p></div>
+                    <?php
+                    wp_admin_notice(
+                        _x( 'Installed', 'theme' ),
+                        array(
+                            'type'               => 'success',
+                            'additional_classes' => array( 'notice-alt' ),
+                        )
+                    );
+                    ?>
                 <# } #>
             <# } else { #>

trunk/src/wp-includes/user.php

@@ -3762 +3762 @@
         $email = get_user_meta( get_current_user_id(), '_new_email', true );
         if ( $email ) {
-            /* translators: %s: New email address. */
-            echo '<div class="notice notice-info"><p>' . sprintf( __( 'Your email address has not been updated yet. Please check your inbox at %s for a confirmation email.' ), '<code>' . esc_html( $email['newemail'] ) . '</code>' ) . '</p></div>';
+            $message = sprintf(
+                /* translators: %s: New email address. */
+                __( 'Your email address has not been updated yet. Please check your inbox at %s for a confirmation email.' ),
+                '<code>' . esc_html( $email['newemail'] ) . '</code>'
+            );
+            wp_admin_notice( $message, array( 'type' => 'info' ) );
         }
     }