WordPress.org
Get WordPress

Make WordPress Core

Changeset 25763


Ignore:
Timestamp:
10/11/2013 04:05:13 PM (13 years ago)
Author:
nacin
Message:

Refine error codes throughout the upgrader so we can better detect at what stage updates fail.

see #22704.

Location:
trunk/src/wp-admin/includes
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/includes/class-wp-upgrader.php

    r25755 r25763  
    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);
    … …  
    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.
    … …  
    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;
    … …  
    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;
    … …  
    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 ) {
    … …  
    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
    … …  
    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);

  • trunk/src/wp-admin/includes/file.php

    r25739 r25763  
    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
    … …  
    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);
    … …  
    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
    … …  
    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
    … …  
    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)
    … …  
    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);
    … …  
    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;
    … …  
    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

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

    r25658 r25763  
    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
    … …  
    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
    … …  
    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 ) )
    … …  
    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
Note: See TracChangeset for help on using the changeset viewer.