Changes from trunk/wp-admin/includes/class-wp-upgrader.php at r18227 to branches/3.1/wp-admin/includes/class-wp-upgrader.php at r17503

File:

• branches/3.1/wp-admin/includes/class-wp-upgrader.php (modified) (16 diffs)

branches/3.1/wp-admin/includes/class-wp-upgrader.php

@@ -26 +26 @@
         var $result = array();
 
+        function WP_Upgrader($skin = null) {
+                return $this->__construct($skin);
+        }
         function __construct($skin = null) {
                 if ( null == $skin )
@@ -209 +212 @@
                 }
 
-                if ( $clear_destination ) {
-                        //We're going to clear the destination if theres something there
-                        $this->skin->feedback('remove_old');
-                        $removed = true;
-                        if ( $wp_filesystem->exists($remote_destination) )
+                if ( $wp_filesystem->exists($remote_destination) ) {
+                        if ( $clear_destination ) {
+                                //We're going to clear the destination if theres something there
+                                $this->skin->feedback('remove_old');
                                 $removed = $wp_filesystem->delete($remote_destination, true);
-                        $removed = apply_filters('upgrader_clear_destination', $removed, $local_destination, $remote_destination, $hook_extra);
-
-                        if ( is_wp_error($removed) )
-                                return $removed;
-                        else if ( ! $removed )
-                                return new WP_Error('remove_old_failed', $this->strings['remove_old_failed']);
-                } elseif ( $wp_filesystem->exists($remote_destination) ) {
-                        //If we're not clearing the destination folder and something exists there allready, Bail.
-                        //But first check to see if there are actually any files in the folder.
-                        $_files = $wp_filesystem->dirlist($remote_destination);
-                        if ( ! empty($_files) ) {
-                                $wp_filesystem->delete($remote_source, true); //Clear out the source files.
-                                return new WP_Error('folder_exists', $this->strings['folder_exists'], $remote_destination );
+                                $removed = apply_filters('upgrader_clear_destination', $removed, $local_destination, $remote_destination, $hook_extra);
+
+                                if ( is_wp_error($removed) )
+                                        return $removed;
+                                else if ( ! $removed )
+                                        return new WP_Error('remove_old_failed', $this->strings['remove_old_failed']);
+                        } else {
+                                //If we're not clearing the destination folder and something exists there allready, Bail.
+                                //But first check to see if there are actually any files in the folder.
+                                $_files = $wp_filesystem->dirlist($remote_destination);
+                                if ( ! empty($_files) ) {
+                                        $wp_filesystem->delete($remote_source, true); //Clear out the source files.
+                                        return new WP_Error('folder_exists', $this->strings['folder_exists'], $remote_destination );
+                                }
                         }
                 }
@@ -300 +303 @@
                 }
 
-                $delete_package = ($download != $package); // Do not delete a "local" file
-
                 //Unzip's the file into a temporary directory
-                $working_dir = $this->unpack_package( $download, $delete_package );
+                $working_dir = $this->unpack_package( $download );
                 if ( is_wp_error($working_dir) ) {
                         $this->skin->error($working_dir);
@@ -581 +582 @@
 
                 // If plugin is in its own directory, recursively delete the directory.
-                if ( strpos($plugin, '/') && $this_plugin_dir != $plugins_dir ) //base check on if plugin includes directory separator AND that its not the root plugin folder
+                if ( strpos($plugin, '/') && $this_plugin_dir != $plugins_dir ) //base check on if plugin includes directory seperator AND that its not the root plugin folder
                         $deleted = $wp_filesystem->delete($this_plugin_dir, true);
                 else
@@ -589 +590 @@
                         return new WP_Error('remove_old_failed', $this->strings['remove_old_failed']);
 
-                return true;
+                return $removed;
         }
 }
@@ -868 +869 @@
 
         function upgrade($current) {
-                global $wp_filesystem, $wp_version;
+                global $wp_filesystem;
 
                 $this->init();
@@ -886 +887 @@
                 $wp_dir = trailingslashit($wp_filesystem->abspath());
 
-                // If partial update is returned from the API, use that, unless we're doing a reinstall.
-                // If we cross the new_bundled version number, then use the new_bundled zip.
-                // Don't though if the constant is set to skip bundled items.
-                // If the API returns a no_content zip, go with it. Finally, default to the full zip.
-                if ( $current->packages->partial && 'reinstall' != $current->response && $wp_version == $current->partial_version )
-                        $to_download = 'partial';
-                elseif ( $current->packages->new_bundled && version_compare( $wp_version, $current->new_bundled, '<' )
-                        && ( ! defined( 'CORE_UPGRADE_SKIP_NEW_BUNDLED' ) || ! CORE_UPGRADE_SKIP_NEW_BUNDLED ) )
-                        $to_download = 'new_bundled';
-                elseif ( $current->packages->no_content )
-                        $to_download = 'no_content';
-                else
-                        $to_download = 'full';
-
-                $download = $this->download_package( $current->packages->$to_download );
+                $download = $this->download_package( $current->package );
                 if ( is_wp_error($download) )
                         return $download;
@@ -937 +924 @@
         var $result = false;
 
+        function WP_Upgrader_Skin($args = array()) {
+                return $this->__construct($args);
+        }
         function __construct($args = array()) {
                 $defaults = array( 'url' => '', 'nonce' => '', 'title' => '', 'context' => false );
@@ -1023 +1013 @@
         var $plugin_network_active = false;
 
+        function Plugin_Upgrader_Skin($args = array()) {
+                return $this->__construct($args);
+        }
+
         function __construct($args = array()) {
                 $defaults = array( 'url' => '', 'plugin' => '', 'nonce' => '', 'title' => __('Update Plugin') );
@@ -1075 +1069 @@
         var $error = false;
 
+        function Bulk_Upgrader_Skin($args = array()) {
+                return $this->__construct($args);
+        }
+
         function __construct($args = array()) {
                 $defaults = array( 'url' => '', 'nonce' => '' );
@@ -1179 +1177 @@
 class Bulk_Plugin_Upgrader_Skin extends Bulk_Upgrader_Skin {
         var $plugin_info = array(); // Plugin_Upgrader::bulk() will fill this in.
-
-        function __construct($args = array()) {
+        function Plugin_Upgrader_Skin($args = array()) {
                 parent::__construct($args);
         }
@@ -1211 +1208 @@
 class Bulk_Theme_Upgrader_Skin extends Bulk_Upgrader_Skin {
         var $theme_info = array(); // Theme_Upgrader::bulk() will fill this in.
-
-        function __construct($args = array()) {
+        function Theme_Upgrader_Skin($args = array()) {
                 parent::__construct($args);
         }
@@ -1254 +1250 @@
         var $type;
 
+        function Plugin_Installer_Skin($args = array()) {
+                return $this->__construct($args);
+        }
+
         function __construct($args = array()) {
                 $defaults = array( 'type' => 'web', 'url' => '', 'plugin' => '', 'nonce' => '', 'title' => '' );
@@ -1318 +1318 @@
         var $type;
 
+        function Theme_Installer_Skin($args = array()) {
+                return $this->__construct($args);
+        }
+
         function __construct($args = array()) {
                 $defaults = array( 'type' => 'web', 'url' => '', 'theme' => '', 'nonce' => '', 'title' => '' );
@@ -1379 +1383 @@
 class Theme_Upgrader_Skin extends WP_Upgrader_Skin {
         var $theme = '';
+
+        function Theme_Upgrader_Skin($args = array()) {
+                return $this->__construct($args);
+        }
 
         function __construct($args = array()) {
@@ -1431 +1439 @@
         var $filename;
 
+        function File_Upload_Upgrader($form, $urlholder) {
+                return $this->__construct($form, $urlholder);
+        }
         function __construct($form, $urlholder) {
                 if ( ! ( ( $uploads = wp_upload_dir() ) && false === $uploads['error'] ) )