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'] ) )