WordPress.org

Make WordPress Core

Ticket #22704: 22704.7.diff

File 22704.7.diff, 11.2 KB (added by nacin, 4 years ago)
  • class-wp-upgrader.php

     
    213213                if ( 1 == count($source_files) && $wp_filesystem->is_dir( trailingslashit($source) . $source_files[0] . '/') ) //Only one folder? Then we want its contents. 
    214214                        $source = trailingslashit($source) . trailingslashit($source_files[0]); 
    215215                elseif ( count($source_files) == 0 ) 
    216                         return new WP_Error( 'incompatible_archive', $this->strings['incompatible_archive'], $this->strings['no_files'] ); //There are no files? 
     216                        return new WP_Error( 'incompatible_archive_empty', $this->strings['incompatible_archive'], $this->strings['no_files'] ); //There are no files? 
    217217                else //It's only a single file, the upgrader will use the foldername of this file as the destination folder. foldername is based on zip filename. 
    218218                        $source = trailingslashit($source); 
    219219 
     
    262262                //Create destination if needed 
    263263                if ( !$wp_filesystem->exists($remote_destination) ) 
    264264                        if ( !$wp_filesystem->mkdir($remote_destination, FS_CHMOD_DIR) ) 
    265                                 return new WP_Error('mkdir_failed', $this->strings['mkdir_failed'], $remote_destination); 
     265                                return new WP_Error('mkdir_failed_destination', $this->strings['mkdir_failed'], $remote_destination); 
    266266 
    267267                // Copy new version of item into place. 
    268268                $result = copy_dir($source, $remote_destination); 
     
    623623                } 
    624624 
    625625                if ( ! $plugins_found ) 
    626                         return new WP_Error( 'incompatible_archive', $this->strings['incompatible_archive'], __('No valid plugins were found.') ); 
     626                        return new WP_Error( 'incompatible_archive_no_plugins', $this->strings['incompatible_archive'], __('No valid plugins were found.') ); 
    627627 
    628628                return $source; 
    629629        } 
     
    989989 
    990990                // A proper archive should have a style.css file in the single subdirectory 
    991991                if ( ! file_exists( $working_directory . 'style.css' ) ) 
    992                         return new WP_Error( 'incompatible_archive', $this->strings['incompatible_archive'], __('The theme is missing the <code>style.css</code> stylesheet.') ); 
     992                        return new WP_Error( 'incompatible_archive_theme_no_style', $this->strings['incompatible_archive'], __('The theme is missing the <code>style.css</code> stylesheet.') ); 
    993993 
    994994                $info = get_file_data( $working_directory . 'style.css', array( 'Name' => 'Theme Name', 'Template' => 'Template' ) ); 
    995995 
    996996                if ( empty( $info['Name'] ) ) 
    997                         return new WP_Error( 'incompatible_archive', $this->strings['incompatible_archive'], __("The <code>style.css</code> stylesheet doesn't contain a valid theme header.") ); 
     997                        return new WP_Error( 'incompatible_archive_theme_no_name', $this->strings['incompatible_archive'], __("The <code>style.css</code> stylesheet doesn't contain a valid theme header.") ); 
    998998 
    999999                // If it's not a child theme, it must have at least an index.php to be legit. 
    10001000                if ( empty( $info['Template'] ) && ! file_exists( $working_directory . 'index.php' ) ) 
    1001                         return new WP_Error( 'incompatible_archive', $this->strings['incompatible_archive'], __('The theme is missing the <code>index.php</code> file.') ); 
     1001                        return new WP_Error( 'incompatible_archive_theme_no_index', $this->strings['incompatible_archive'], __('The theme is missing the <code>index.php</code> file.') ); 
    10021002 
    10031003                return $source; 
    10041004        } 
     
    11411141                $remote_destination = $wp_filesystem->find_folder( WP_LANG_DIR ); 
    11421142                if ( ! $wp_filesystem->exists( $remote_destination ) ) 
    11431143                        if ( ! $wp_filesystem->mkdir( $remote_destination, FS_CHMOD_DIR ) ) 
    1144                                 return new WP_Error( 'mkdir_failed', $this->strings['mkdir_failed'], $remote_destination ); 
     1144                                return new WP_Error( 'mkdir_failed_lang_dir', $this->strings['mkdir_failed'], $remote_destination ); 
    11451145 
    11461146                foreach ( $language_updates as $language_update ) { 
    11471147 
     
    12001200                } 
    12011201 
    12021202                if ( ! $mo || ! $po ) 
    1203                         return new WP_Error( 'incompatible_archive', $this->strings['incompatible_archive'], 
     1203                        return new WP_Error( 'incompatible_archive_pomo', $this->strings['incompatible_archive'], 
    12041204                                __( 'The language pack is missing either the <code>.po</code> or <code>.mo</code> files.' ) ); 
    12051205 
    12061206                return $source; 
     
    12871287                // Copy update-core.php from the new version into place. 
    12881288                if ( !$wp_filesystem->copy($working_dir . '/wordpress/wp-admin/includes/update-core.php', $wp_dir . 'wp-admin/includes/update-core.php', true) ) { 
    12891289                        $wp_filesystem->delete($working_dir, true); 
    1290                         return new WP_Error('copy_failed', $this->strings['copy_failed']); 
     1290                        return new WP_Error('copy_failed_for_update_core_file', $this->strings['copy_failed']); 
    12911291                } 
    12921292                $wp_filesystem->chmod($wp_dir . 'wp-admin/includes/update-core.php', FS_CHMOD_FILE); 
    12931293 
  • file.php

     
    603603 
    604604        for ( $i = 0; $i < $z->numFiles; $i++ ) { 
    605605                if ( ! $info = $z->statIndex($i) ) 
    606                         return new WP_Error('stat_failed', __('Could not retrieve file from archive.')); 
     606                        return new WP_Error('stat_failed_ziparchive', __('Could not retrieve file from archive.')); 
    607607 
    608608                if ( '__MACOSX/' === substr($info['name'], 0, 9) ) // Skip the OS X-created __MACOSX directory 
    609609                        continue; 
     
    633633        // Create those directories if need be: 
    634634        foreach ( $needed_dirs as $_dir ) { 
    635635                if ( ! $wp_filesystem->mkdir($_dir, FS_CHMOD_DIR) && ! $wp_filesystem->is_dir($_dir) ) // Only check to see if the Dir exists upon creation failure. Less I/O this way. 
    636                         return new WP_Error('mkdir_failed', __('Could not create directory.'), $_dir); 
     636                        return new WP_Error('mkdir_failed_ziparchive', __('Could not create directory.'), $_dir); 
    637637        } 
    638638        unset($needed_dirs); 
    639639 
    640640        for ( $i = 0; $i < $z->numFiles; $i++ ) { 
    641641                if ( ! $info = $z->statIndex($i) ) 
    642                         return new WP_Error('stat_failed', __('Could not retrieve file from archive.')); 
     642                        return new WP_Error('stat_failed_ziparchive', __('Could not retrieve file from archive.')); 
    643643 
    644644                if ( '/' == substr($info['name'], -1) ) // directory 
    645645                        continue; 
     
    649649 
    650650                $contents = $z->getFromIndex($i); 
    651651                if ( false === $contents ) 
    652                         return new WP_Error('extract_failed', __('Could not extract file from archive.'), $info['name']); 
     652                        return new WP_Error('extract_failed_ziparchive', __('Could not extract file from archive.'), $info['name']); 
    653653 
    654654                if ( ! $wp_filesystem->put_contents( $to . $info['name'], $contents, FS_CHMOD_FILE) ) 
    655                         return new WP_Error('copy_failed', __('Could not copy file.'), $to . $info['name']); 
     655                        return new WP_Error('copy_failed_ziparchive', __('Could not copy file.'), $to . $info['name']); 
    656656        } 
    657657 
    658658        $z->close(); 
     
    691691                return new WP_Error('incompatible_archive', __('Incompatible Archive.'), $archive->errorInfo(true)); 
    692692 
    693693        if ( 0 == count($archive_files) ) 
    694                 return new WP_Error('empty_archive', __('Empty archive.')); 
     694                return new WP_Error('empty_archive_pclzip', __('Empty archive.')); 
    695695 
    696696        // Determine any children directories needed (From within the archive) 
    697697        foreach ( $archive_files as $file ) { 
     
    720720        // Create those directories if need be: 
    721721        foreach ( $needed_dirs as $_dir ) { 
    722722                if ( ! $wp_filesystem->mkdir($_dir, FS_CHMOD_DIR) && ! $wp_filesystem->is_dir($_dir) ) // Only check to see if the dir exists upon creation failure. Less I/O this way. 
    723                         return new WP_Error('mkdir_failed', __('Could not create directory.'), $_dir); 
     723                        return new WP_Error('mkdir_failed_pclzip', __('Could not create directory.'), $_dir); 
    724724        } 
    725725        unset($needed_dirs); 
    726726 
     
    733733                        continue; 
    734734 
    735735                if ( ! $wp_filesystem->put_contents( $to . $file['filename'], $file['content'], FS_CHMOD_FILE) ) 
    736                         return new WP_Error('copy_failed', __('Could not copy file.'), $to . $file['filename']); 
     736                        return new WP_Error('copy_failed_pclzip', __('Could not copy file.'), $to . $file['filename']); 
    737737        } 
    738738        return true; 
    739739} 
     
    766766                                // If copy failed, chmod file to 0644 and try again. 
    767767                                $wp_filesystem->chmod($to . $filename, 0644); 
    768768                                if ( ! $wp_filesystem->copy($from . $filename, $to . $filename, true, FS_CHMOD_FILE) ) 
    769                                         return new WP_Error('copy_failed', __('Could not copy file.'), $to . $filename); 
     769                                        return new WP_Error('copy_failed_copy_dir', __('Could not copy file.'), $to . $filename); 
    770770                        } 
    771771                } elseif ( 'd' == $fileinfo['type'] ) { 
    772772                        if ( !$wp_filesystem->is_dir($to . $filename) ) { 
    773773                                if ( !$wp_filesystem->mkdir($to . $filename, FS_CHMOD_DIR) ) 
    774                                         return new WP_Error('mkdir_failed', __('Could not create directory.'), $to . $filename); 
     774                                        return new WP_Error('mkdir_failed_copy_dir', __('Could not create directory.'), $to . $filename); 
    775775                        } 
    776776 
    777777                        // generate the $sub_skip_list for the subdirectory as a sub-set of the existing $skip_list 
  • update-core.php

     
    661661        $versions_file = trailingslashit( $wp_filesystem->wp_content_dir() ) . 'upgrade/version-current.php'; 
    662662        if ( ! $wp_filesystem->copy( $from . $distro . 'wp-includes/version.php', $versions_file ) ) { 
    663663                 $wp_filesystem->delete( $from, true ); 
    664                  return new WP_Error( 'copy_failed', __('Could not copy file.') ); 
     664                 return new WP_Error( 'copy_failed_for_version_file', __('Could not copy file.') ); 
    665665        } 
    666666 
    667667        $wp_filesystem->chmod( $versions_file, FS_CHMOD_FILE ); 
     
    741741                        $total_size += filesize( $working_dir_local . '/' . $file );  
    742742 
    743743                // If we don't have enough free space, it isn't worth trying again 
    744                 if ( $total_size >= disk_free_space( ABSPATH ) ) 
     744                if ( $total_size >= disk_free_space( ABSPATH ) ) { 
    745745                        $result = new WP_Error( 'disk_full', __( 'There is not enough free disk space to complete the update.' ), $to ); 
    746                 else 
     746                } else { 
    747747                        $result = _copy_dir( $from . $distro, $to, $skip ); 
     748                        if ( is_wp_error( $result ) ) 
     749                                $result = new WP_Error( $result->get_error_code() . '_retry', $result->get_error_message(), $result->get_error_data() ); 
     750                } 
    748751        } 
    749752 
    750753        // Custom Content Directory needs updating now. 
     
    800803                                                continue; 
    801804 
    802805                                        if ( ! $wp_filesystem->copy($from . $distro . 'wp-content/' . $file, $dest . $filename, FS_CHMOD_FILE) ) 
    803                                                 $result = new WP_Error('copy_failed', __('Could not copy file.'), $dest . $filename); 
     806                                                $result = new WP_Error('copy_failed_for_new_bundled', __('Could not copy file.'), $dest . $filename); 
    804807                                } else { 
    805808                                        if ( ! $development_build && $wp_filesystem->is_dir( $dest . $filename ) ) 
    806809                                                continue; 
     
    887890                                // If copy failed, chmod file to 0644 and try again. 
    888891                                $wp_filesystem->chmod($to . $filename, 0644); 
    889892                                if ( ! $wp_filesystem->copy($from . $filename, $to . $filename, true, FS_CHMOD_FILE) ) 
    890                                         return new WP_Error('copy_failed', __('Could not copy file.'), $to . $filename); 
     893                                        return new WP_Error('copy_failed__copy_dir', __('Could not copy file.'), $to . $filename); 
    891894                        } 
    892895                } elseif ( 'd' == $fileinfo['type'] ) { 
    893896                        if ( !$wp_filesystem->is_dir($to . $filename) ) { 
    894897                                if ( !$wp_filesystem->mkdir($to . $filename, FS_CHMOD_DIR) ) 
    895                                         return new WP_Error('mkdir_failed', __('Could not create directory.'), $to . $filename); 
     898                                        return new WP_Error('mkdir_failed__copy_dir', __('Could not create directory.'), $to . $filename); 
    896899                        } 
    897900 
    898901                        // generate the $sub_skip_list for the subdirectory as a sub-set of the existing $skip_list