WordPress.org

Make WordPress Core

Ticket #5586: 5586.9.diff

File 5586.9.diff, 9.6 KB (added by DD32, 13 years ago)

  • wp-admin/includes/class-wp-filesystem-direct.php

     
    271271                foreach($filelist as $filename=>$det){
    272272                        if ( '/' == substr($filename,-1,1) )
    273273                                $this->rmdir($path.'/'.$filename,$recursive);
    274                         @rmdir($entry);
     274                        @rmdir($filename);
    275275                }
    276276                return @rmdir($path);
    277277        }
    … …  
    333333                return;
    334334        }
    335335}
    336 ?>
    337  No newline at end of file
     336?>

  • wp-admin/includes/class-wp-filesystem-ftpext.php

     
    132132        }
    133133        function get_contents($file,$type='',$resumepos=0){
    134134                if( empty($type) ){
    135                         $extension = substr(strrchr($filename, "."), 1);
     135                        $extension = substr(strrchr($file, "."), 1);
    136136                        $type = isset($this->filetypes[ $extension ]) ? $this->filetypes[ $extension ] : FTP_ASCII;
    137137                }
    138138                $temp = tmpfile();
    … …  
    348348        }
    349349        function rmdir($path,$recursive=false){
    350350                if( ! $recursive )
    351                         return @ftp_rmdir($this->link, $file);
     351                        return @ftp_rmdir($this->link, $path);
    352352
    353353                //TODO: Recursive Directory delete, Have to delete files from the folder first.
    354354                //$dir = $this->dirlist($path);

  • wp-admin/includes/class-wp-filesystem-ftpsockets.php

     
    139139                        return false;
    140140
    141141                if( empty($type) ){
    142                         $extension = substr(strrchr($filename, "."), 1);
     142                        $extension = substr(strrchr($file, "."), 1);
    143143                        $type = isset($this->filetypes[ $extension ]) ? $this->filetypes[ $extension ] : FTP_AUTOASCII;
    144144                }
    145145                $this->ftp->SetType($type);
    … …  
    376376
    377377        function rmdir($path,$recursive=false){
    378378                if( ! $recursive )
    379                         return $this->ftp->rmdir($file);
     379                        return $this->ftp->rmdir($path);
    380380
    381381                return $this->ftp->mdel($path);
    382382        }

  • wp-admin/includes/file.php

     
    231231
    232232        // Is the archive valid?
    233233        if ( false == ($archive_files = $archive->extract(PCLZIP_OPT_EXTRACT_AS_STRING)) )
    234                 return new WP_Error('incompatible_archive', __('Incompatible archive'), $archive->error_string);
     234                return new WP_Error('incompatible_archive', __('Incompatible archive'), $archive->errorInfo(true));
    235235
    236236        if ( 0 == count($archive_files) )
    237237                return new WP_Error('empty_archive', __('Empty archive'));
    … …  
    240240        $path = explode('/', $to);
    241241        $tmppath = '';
    242242        for ( $j = 0; $j < count($path) - 1; $j++ ) {
    243                 $prevpath = $tmppath;
    244243                $tmppath .= $path[$j] . '/';
    245                 if ( ! $fs->is_dir($tmppath) ) {
    246                         //$fs->setDefaultPermissions( $fs->getchmod($tmppath) );
     244                if ( ! $fs->is_dir($tmppath) )
    247245                        $fs->mkdir($tmppath, 0755);
    248                 }
    249246        }
    250247
    251248        foreach ($archive_files as $file) {

  • wp-admin/includes/update.php

     
    114114        $r = $current->response[ $file ];
    115115
    116116        echo "<tr><td colspan='5' class='plugin-update'>";
    117         printf( __('There is a new version of %1$s available. <a href="%2$s">Download version %3$s here</a> or <a href="%4$s">upgrade automatically</a>.'), $plugin_data['Name'], $r->url, $r->new_version, "update.php?action=upgrade-plugin&amp;plugin=$file" );
     117        printf( __('There is a new version of %1$s available. <a href="%2$s">Download version %3$s here</a> or <a href="%4$s">upgrade automatically</a>.'), $plugin_data['Name'], $r->url, $r->new_version, wp_nonce_url("update.php?action=upgrade-plugin&amp;plugin=$file", 'upgrade-plugin_' . $file) );
    118118        echo "</td></tr>";
    119119}
    120120add_action( 'after_plugin_row', 'wp_plugin_update_row' );
    … …  
    154154
    155155        // Download the package
    156156        $package = $r->package;
    157         apply_filters('update_feedback', sprintf(__("Downloading update from %s"), $package));
     157        apply_filters('update_feedback', sprintf(__('Downloading update from %s'), $package));
    158158        $file = download_url($package);
    159159
    160160        if ( !$file )
    161161                return new WP_Error('download_failed', __('Download failed.'));
    162162
    163         $name = basename($plugin, '.php');
    164         $working_dir = $base . 'wp-content/upgrade/' . $name;
     163        $working_dir = $base . 'wp-content/upgrade/' . basename($plugin, '.php');
    165164
    166165        // Clean up working directory
    167         if ( is_dir($working_dir) )
     166        if ( $wp_filesystem->is_dir($working_dir) )
    168167                $wp_filesystem->delete($working_dir, true);
    169168
    170         apply_filters('update_feedback', __("Unpacking the update"));
     169        apply_filters('update_feedback', __('Unpacking the update'));
    171170        // Unzip package to working directory
    172171        $result = unzip_file($file, $working_dir);
    173172        if ( is_wp_error($result) ) {
    … …  
    180179        unlink($file);
    181180
    182181        // Remove the existing plugin.
    183         apply_filters('update_feedback', __("Removing the old version of the plugin"));
     182        apply_filters('update_feedback', __('Removing the old version of the plugin'));
    184183        $plugin_dir = dirname($base . PLUGINDIR . "/$plugin");
    185184        $plugin_dir = trailingslashit($plugin_dir);
     185       
    186186        // If plugin is in its own directory, recursively delete the directory.
    187         if( ! in_array( $plugin_dir, array('.', trailingslashit($base . PLUGINDIR) ) ) )
     187        if( strpos($plugin, '/') && $plugin_dir != $base . PLUGINDIR . '/' )
    188188                $deleted = $wp_filesystem->delete($plugin_dir, true);
    189189        else
    190190                $deleted = $wp_filesystem->delete($base . PLUGINDIR . "/$plugin");
     191
    191192        if ( !$deleted ) {
    192193                $wp_filesystem->delete($working_dir, true);
    193194                return new WP_Error('delete_failed', __('Could not remove the old plugin'));
    194195        }
    195196
    196         apply_filters('update_feedback', __("Installing the latest version"));
     197        apply_filters('update_feedback', __('Installing the latest version'));
    197198        // Copy new version of plugin into place.
    198199        if ( !copy_dir($working_dir, $base . PLUGINDIR) ) {
    199                 //$wp_filesystem->delete($working_dir, true);
     200                //$wp_filesystem->delete($working_dir, true); //TODO: Uncomment? This DOES mean that the new files are available in the upgrade folder if it fails.
    200201                return new WP_Error('install_failed', __('Installation failed'));
    201202        }
    202203
    … …  
    207208        delete_option('update_plugins');
    208209}
    209210
    210 ?>
     211?>
     212 No newline at end of file

  • wp-admin/update.php

     
    55if ( !current_user_can('edit_plugins') )
    66                wp_die('<p>'.__('You do not have sufficient permissions to update plugins for this blog.').'</p>');
    77
    8 function request_filesystem_credentials($form_post, $type = '') {
     8function request_filesystem_credentials($form_post, $type = '', $error = false) {
    99        if ( empty($type) )
    1010                $type = get_filesystem_method();
    1111
    1212        if ( 'direct' == $type )
    1313                return array();
    1414
    15         if ( !empty($_POST['password']) && !empty($_POST['username']) && !empty($_POST['hostname']) ) {
     15        if ( ! $error && !empty($_POST['password']) && !empty($_POST['username']) && !empty($_POST['hostname']) ) {
    1616                $credentials = array('hostname' => $_POST['hostname'], 'username' => $_POST['username'],
    1717                        'password' => $_POST['password'], 'ssl' => $_POST['ssl']);
    1818                $stored_credentials = $credentials;
    … …  
    2626        $ssl = '';
    2727        if ( $credentials = get_option('ftp_credentials') )
    2828                extract($credentials, EXTR_OVERWRITE);
     29        if( $error ){
     30                echo '<div id="message" class="error"><p>' . __('<strong>Error:</strong> There was an error connecting to the server, Please verify the settings are correct.') . '</p></div>';
     31        }
    2932?>
    3033<form action="<?php echo $form_post ?>" method="post">
    3134<div class="wrap">
    … …  
    4245</tr>
    4346<tr valign="top">
    4447<th scope="row"><?php _e('Password:') ?></th>
    45 <td><input name="password" type="text" id="password" value="<?php echo attribute_escape($password) ?>" size="40" /></td>
     48<td><input name="password" type="password" id="password" value="<?php echo attribute_escape($password) ?>" size="40" /></td>
    4649</tr>
    4750<tr valign="top">
    4851<th scope="row"><?php _e('Use SSL:') ?></th>
    … …  
    8083function do_plugin_upgrade($plugin) {
    8184        global $wp_filesystem;
    8285
    83         $credentials = request_filesystem_credentials("update.php?action=upgrade-plugin&plugin=$plugin");
    84         if ( false === $credentials )
     86        $url = wp_nonce_url("update.php?action=upgrade-plugin&plugin=$plugin", "upgrade-plugin_$plugin");
     87        if ( false === ($credentials = request_filesystem_credentials($url)) )
    8588                return;
     89               
     90        if( ! WP_Filesystem($credentials) ){
     91                request_filesystem_credentials($url, '', true); //Failed to connect, Error and request again
     92                return;
     93        }
     94               
    8695        echo '<div class="wrap">';
    8796        echo '<h2>' . __('Upgrade Plugin') . '</h2>';
    88         WP_Filesystem($credentials);
    89         // TODO: look for auth and connect error codes and direct back to credentials form.
    9097        if ( $wp_filesystem->errors->get_error_code() ) {
    9198                foreach ( $wp_filesystem->errors->get_error_messages() as $message )
    9299                        show_message($message);
    … …  
    108115                $plugin = trim($_GET['plugin']);
    109116
    110117        if ( 'upgrade-plugin' == $_GET['action'] ) {
    111                 //check-admin_referer('upgrade-plugin_' . $plugin);
     118                check_admin_referer('upgrade-plugin_' . $plugin);
    112119                $title = __('Upgrade Plugin');
    113120                $parent_file = 'plugins.php';
    114121                require_once('admin-header.php');
    115122                do_plugin_upgrade($plugin);
    116123                include('admin-footer.php');
    117124        }
    118 
    119125}
    120126
    121 ?>
    122  No newline at end of file
     127?>