WordPress.org

Make WordPress Core

Changeset 25772


Ignore:
Timestamp:
10/14/13 19:37:53 (4 years ago)
Author:
nacin
Message:

More specific error codes in the core upgrader when copying language, plugin, and theme files. see #22704.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/update-core.php

    r25763 r25772  
    766766        if ( @is_dir($lang_dir) ) { 
    767767            $wp_lang_dir = $wp_filesystem->find_folder($lang_dir); 
    768             if ( $wp_lang_dir ) 
     768            if ( $wp_lang_dir ) { 
    769769                $result = copy_dir($from . $distro . 'wp-content/languages/', $wp_lang_dir); 
     770                if ( is_wp_error( $result ) ) 
     771                    $result = new WP_Error( $result->get_error_code() . '_languages', $result->get_error_message(), $result->get_error_data() ); 
     772            } 
    770773        } 
    771774    } 
     
    804807 
    805808                    if ( ! $wp_filesystem->copy($from . $distro . 'wp-content/' . $file, $dest . $filename, FS_CHMOD_FILE) ) 
    806                         $result = new WP_Error( 'copy_failed_for_new_bundled', __( 'Could not copy file.' ), $dest . $filename ); 
     809                        $result = new WP_Error( "copy_failed_for_new_bundled_$type", __( 'Could not copy file.' ), $dest . $filename ); 
    807810                } else { 
    808811                    if ( ! $development_build && $wp_filesystem->is_dir( $dest . $filename ) ) 
     
    811814                    $wp_filesystem->mkdir($dest . $filename, FS_CHMOD_DIR); 
    812815                    $_result = copy_dir( $from . $distro . 'wp-content/' . $file, $dest . $filename); 
    813                     if ( is_wp_error($_result) ) //If a error occurs partway through this final step, keep the error flowing through, but keep process going. 
    814                         $result = $_result; 
     816 
     817                    // If a error occurs partway through this final step, keep the error flowing through, but keep process going. 
     818                    if ( is_wp_error( $_result ) ) { 
     819                        if ( ! is_wp_error( $result ) ) 
     820                            $result = new WP_Error; 
     821                        $result->add( $_result->get_error_code() . "_$type", $_result->get_error_message(), $_result->get_error_data() ); 
     822                    } 
    815823                } 
    816824            } 
Note: See TracChangeset for help on using the changeset viewer.