WordPress.org

Make WordPress Core

Changeset 25772


Ignore:
Timestamp:
10/14/2013 07:37:53 PM (5 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.