Ticket #51857: 51857.2.diff

wp-admin/includes/file.php

@@ -1730 +1730 @@
  * Assumes that WP_Filesystem() has already been called and setup.
  *
  * @since 2.5.0
+ * @since 5.7.0 Add $first_pass parameter.
  *
  * @global WP_Filesystem_Base $wp_filesystem WordPress filesystem subclass.
  *
- * @param string   $from      Source directory.
- * @param string   $to        Destination directory.
- * @param string[] $skip_list An array of files/folders to skip copying.
+ * @param string   $from       Source directory.
+ * @param string   $to         Destination directory.
+ * @param string[] $skip_list  An array of files/folders to skip copying.
+ * @param bool     $first_pass True on the initial call, but false on subsequent calls.
  * @return true|WP_Error True on success, WP_Error on failure.
  */
-function copy_dir( $from, $to, $skip_list = array() ) {
+function copy_dir( $from, $to, $skip_list = array(), $first_pass = true ) {
         global $wp_filesystem;
 
         $dirlist = $wp_filesystem->dirlist( $from );
 
+        if ( ! $dirlist && $first_pass ) {
+                return new WP_Error( 'dirlist_failed_copy_dir', __( 'Directory listing failed.' ), $wp_filesystem->method );
+        }
+
         $from = trailingslashit( $from );
         $to   = trailingslashit( $to );
 
@@ -1776 +1782 @@
                                 }
                         }
 
-                        $result = copy_dir( $from . $filename, $to . $filename, $sub_skip_list );
+                        $result = copy_dir( $from . $filename, $to . $filename, $sub_skip_list, false );
                         if ( is_wp_error( $result ) ) {
                                 return $result;
                         }