Changeset 48445 for trunk/src/wp-admin/includes/ajax-actions.php

Timestamp:

07/12/2020 01:14:01 AM (6 years ago)

Author:

SergeyBiryukov

Message:

Administration: Handle the result of Plugin_Upgrader::bulk_upgrade() for a plugin that is already at the latest version in the same way it is handled for themes.

This corrects a fragile check of the result in wp_ajax_update_plugin() that depended on the internal array pointer, and brings some consistency with wp_ajax_update_theme().

Follow-up to [37714], [48401].
See #50448.

File:

• trunk/src/wp-admin/includes/ajax-actions.php (modified) (2 diffs)

trunk/src/wp-admin/includes/ajax-actions.php

@@ -4463 +4463 @@
                 wp_send_json_error( $status );
         } elseif ( is_array( $result ) && ! empty( $result[ $plugin ] ) ) {
-                $plugin_update_data = current( $result );
 
                 /*
-                 * If the `update_plugins` site transient is empty (e.g. when you update
-                 * two plugins in quick succession before the transient repopulates),
-                 * this may be the return.
+                 * Plugin is already at the latest version.
+                 *
+                 * This may also be the return value If the `update_plugins` site transient is empty,
+                 * e.g. when you update two plugins in quick succession before the transient repopulates.
                  *
                  * Preferably something can be done to ensure `update_plugins` isn't empty.
                  * For now, surface some sort of error here.
                  */
-                if ( true === $plugin_update_data ) {
-                        $status['errorMessage'] = __( 'Plugin update failed.' );
+                if ( true === $result[ $plugin ] ) {
+                        $status['errorMessage'] = $upgrader->strings['up_to_date'];
                         wp_send_json_error( $status );
                 }
@@ -4485 +4485 @@
                         $status['newVersion'] = sprintf( __( 'Version %s' ), $plugin_data['Version'] );
                 }
+
                 wp_send_json_success( $status );
         } elseif ( false === $result ) {