Changeset 43097

Timestamp:

05/02/2018 02:37:50 AM (7 years ago)

Author:

SergeyBiryukov

Message:

Privacy: Include wp-admin/includes/file.php to avoid fatal error.

list_files() is defined in wp-admin/includes/file.php, which is not included by wp-cron.php, so it needs to be included by the caller in order to avoid a fatal PHP error.

This bug was not detected during testing because the file _is_ included when executing jobs via wp cron event run.

Props mikejolley, iandunn.
Merges [43059] to the 4.9 branch.
See #43546.
See ​https://wordpress.slack.com/archives/C9695RJBW/p1525190405000860.

Location:

branches/4.9

Files:

• . (modified) (1 prop)
• src/wp-includes/functions.php (modified) (1 diff)

branches/4.9/src/wp-includes/functions.php

@@ -5959 +5959 @@
  */
 function wp_privacy_delete_old_export_files() {
+    require_once( ABSPATH . 'wp-admin/includes/file.php' );
+
     $upload_dir   = wp_upload_dir();
     $exports_dir  = trailingslashit( $upload_dir['basedir'] . '/exports' );