Changeset 56599 for trunk/src/wp-admin/includes/class-bulk-upgrader-skin.php

Timestamp:

09/17/2023 03:21:07 PM (3 years 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/class-bulk-upgrader-skin.php (modified) (1 diff)

trunk/src/wp-admin/includes/class-bulk-upgrader-skin.php

@@ -144 +144 @@
         if ( $this->error || ! $this->result ) {
             if ( $this->error ) {
-                echo '<div class="error"><p>' . sprintf( $this->upgrader->strings['skin_update_failed_error'], $title, '<strong>' . $this->error . '</strong>' ) . '</p></div>';
+                $after_error_message = sprintf( $this->upgrader->strings['skin_update_failed_error'], $title, '<strong>' . $this->error . '</strong>' );
             } else {
-                echo '<div class="error"><p>' . sprintf( $this->upgrader->strings['skin_update_failed'], $title ) . '</p></div>';
+                $after_error_message = sprintf( $this->upgrader->strings['skin_update_failed'], $title );
             }
+            wp_admin_notice(
+                $after_error_message,
+                array(
+                    'additional_classes' => array( 'error' ),
+                )
+            );
 
             echo '<script type="text/javascript">jQuery(\'#progress-' . esc_js( $this->upgrader->update_current ) . '\').show();</script>';