Changeset 52192 for trunk/src/wp-admin/includes/class-wp-upgrader.php

Timestamp:

11/16/2021 10:18:32 PM (3 years ago)

Author:

SergeyBiryukov

Message:

Upgrade/Install: Correct the weekly cron event for clearing the temp-backup directory:

• Make sure the wp_delete_temp_updater_backups event has an action associated with it when it runs.
• Check if the cron event already exists before scheduling it, to avoid scheduling duplicate events.
• Move the code for clearing the temp-backup directory to a standalone function.

Follow-up to [51815], [51898], [51899].

Props pbiron, johnbillion.
See #51857.

File:

• trunk/src/wp-admin/includes/class-wp-upgrader.php (modified) (2 diffs)

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

@@ -151 +151 @@
      */
     protected function schedule_temp_backup_cleanup() {
-        wp_schedule_event( time(), 'weekly', 'delete_temp_updater_backups' );
-        add_action( 'delete_temp_updater_backups', array( $this, 'delete_all_temp_backups' ) );
+        if ( false === wp_next_scheduled( 'wp_delete_temp_updater_backups' ) ) {
+            wp_schedule_event( time(), 'weekly', 'wp_delete_temp_updater_backups' );
+        }
     }
 
@@ -1108 +1109 @@
         );
     }
-
-    /**
-     * Deletes all contents of the temp-backup directory.
-     *
-     * @since 5.9.0
-     *
-     * @global WP_Filesystem_Base $wp_filesystem WordPress filesystem subclass.
-     */
-    public function delete_all_temp_backups() {
-        /*
-         * Check if there's a lock, or if currently performing an Ajax request,
-         * in which case there's a chance we're doing an update.
-         * Reschedule for an hour from now and exit early.
-         */
-        if ( get_option( 'core_updater.lock' ) || get_option( 'auto_updater.lock' ) || wp_doing_ajax() ) {
-            wp_schedule_single_event( time() + HOUR_IN_SECONDS, 'delete_temp_updater_backups' );
-            return;
-        }
-
-        add_action(
-            'shutdown',
-            /*
-             * This action runs on shutdown to make sure there's no plugin updates currently running.
-             * Using a closure in this case is OK since the action can be removed by removing the parent hook.
-             */
-            function() {
-                global $wp_filesystem;
-
-                if ( ! $wp_filesystem ) {
-                    include_once ABSPATH . '/wp-admin/includes/file.php';
-                    WP_Filesystem();
-                }
-
-                $dirlist = $wp_filesystem->dirlist( $wp_filesystem->wp_content_dir() . 'upgrade/temp-backup/' );
-
-                foreach ( array_keys( $dirlist ) as $dir ) {
-                    if ( '.' === $dir || '..' === $dir ) {
-                        continue;
-                    }
-
-                    $wp_filesystem->delete( $wp_filesystem->wp_content_dir() . 'upgrade/temp-backup/' . $dir, true );
-                }
-            }
-        );
-    }
 }