Changeset 56572

Timestamp:

09/14/2023 01:23:37 AM (3 years 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' ) );
                 }
         }