WordPress.org

Make WordPress Core

Ticket #5586: 5586.4.diff

File 5586.4.diff, 5.1 KB (added by DD32, 13 years ago)

Better WP location search / Allow blank files in installer

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

     
    8585        }
    8686
    8787        function find_base_dir($base = '.',$echo = false){
     88                $abspath = str_replace('\\','/',ABSPATH); //windows: Straighten up the paths..
     89                if( strpos($abspath, ':') ){ //Windows, Strip out the driveletter
     90                        if( preg_match("|.{1}\:(.+)|i", $abspath, $mat) )
     91                                $abspath = $mat[1];
     92                }
     93       
    8894                if( empty( $base ) || '.' == $base ) $base = $this->cwd();
    8995                if( empty( $base ) ) $base = '/';
    9096                if( '/' != substr($base, -1) ) $base .= '/';
    9197
    92                 if($echo) echo sprintf(__('Changing to %s'), $base) .'<br>';
    93                 if( false === ftp_chdir($this->link, $base) )
     98                if($echo) echo __('Changing to ') . $base .'<br>';
     99                if( false === $this->chdir($base) )
    94100                        return false;
    95101
    96102                if( $this->exists($base . 'wp-settings.php') ){
    97                         if($echo) echo sprintf(__('Found %s'), $base . 'wp-settings.php') . '<br>';
     103                        if($echo) echo __('Found ') . $base . 'wp-settings.php<br>';
    98104                        $this->wp_base = $base;
    99105                        return $this->wp_base;
    100106                }
    101107
    102                 if( strpos(ABSPATH, $base) > 0)
    103                         $arrPath = split('/',substr(ABSPATH,strpos(ABSPATH, $base)));
     108                if( strpos($abspath, $base) > 0)
     109                        $arrPath = split('/',substr($abspath,strpos($abspath, $base)));
    104110                else
    105                         $arrPath = split('/',ABSPATH);
     111                        $arrPath = split('/',$abspath);
    106112
    107113                for($i = 0; $i <= count($arrPath); $i++)
    108114                        if( $arrPath[ $i ] == '' ) unset( $arrPath[ $i ] );
    109115
    110116                foreach($arrPath as $key=>$folder){
    111117                        if( $this->is_dir($base . $folder) ){
    112                                 if($echo) echo sprintf(__('Found %s'), $folder) . ' ' . sprintf(__('Changing to %s'), $base . $folder . '/') . '<br>';
     118                                if($echo) echo __('Found ') . $folder . ' ' . __('Changing to') . ' ' . $base . $folder . '/<br>';
    113119                                return $this->find_base_dir($base .  $folder . '/',$echo);
    114120                        }
    115121                }
    … …  
    158164        function cwd(){
    159165                return ftp_pwd($this->link);
    160166        }
     167        function chdir($dir){
     168                return @ftp_chdir($dir);
     169        }
    161170        function chgrp($file,$group,$recursive=false){
    162171                return false;
    163172        }
    … …  
    170179                        return false;
    171180                if ( ! $recursive || ! $this->is_dir($file) ){
    172181                        if (!function_exists('ftp_chmod'))
    173                                 return ftp_site($this->link, sprintf('CHMOD %o %s', $mode, $file));
    174                         return ftp_chmod($this->link,$mode,$file);
     182                                return @ftp_site($this->link, sprintf('CHMOD %o %s', $mode, $file));
     183                        return @ftp_chmod($this->link,$mode,$file);
    175184                }
    176185                //Is a directory, and we want recursive
    177186                $filelist = $this->dirlist($file);
    … …  
    267276        function copy($source,$destination,$overwrite=false){
    268277                if( ! $overwrite && $this->exists($destination) )
    269278                        return false;
    270                 if ( !$content = $this->get_contents($source) )
     279                $content = $this->get_contents($source);
     280                if( false === $content)
    271281                        return false;
    272282                return $this->put_contents($destination,$content);
    273283        }

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

     
    8787        }
    8888
    8989        function find_base_dir($base = '.',$echo = false) {
     90                $abspath = str_replace('\\','/',ABSPATH); //windows: Straighten up the paths..
     91                if( strpos($abspath, ':') ){ //Windows, Strip out the driveletter
     92                        if( preg_match("|.{1}\:(.+)|i", $abspath, $mat) )
     93                                $abspath = $mat[1];
     94                }
     95       
    9096                if( empty( $base ) || '.' == $base ) $base = $this->cwd();
    9197                if( empty( $base ) ) $base = '/';
    9298                if( '/' != substr($base, -1) ) $base .= '/';
    9399
    94100                if($echo) echo __('Changing to ') . $base  .'<br>';
    95                 if( false === $this->ftp->chdir($base) )
     101                if( false === $this->chdir($base) )
    96102                        return false;
    97103
    98104                if( $this->exists($base . 'wp-settings.php') ){
    … …  
    101107                        return $this->wp_base;
    102108                }
    103109
    104                 if( strpos(ABSPATH, $base) > 0)
    105                         $arrPath = split('/',substr(ABSPATH,strpos(ABSPATH, $base)));
     110                if( strpos($abspath, $base) > 0)
     111                        $arrPath = split('/',substr($abspath,strpos($abspath, $base)));
    106112                else
    107                         $arrPath = split('/',ABSPATH);
     113                        $arrPath = split('/',$abspath);
    108114
    109115                for($i = 0; $i <= count($arrPath); $i++)
    110116                        if( $arrPath[ $i ] == '' ) unset( $arrPath[ $i ] );
    … …  
    129135        }
    130136
    131137        function get_contents($file,$type='',$resumepos=0){
     138                if( ! $this->exists($file) )
     139                        return false;
     140
    132141                if( empty($type) ){
    133142                        $extension = substr(strrchr($filename, "."), 1);
    134143                        $type = isset($this->filetypes[ $extension ]) ? $this->filetypes[ $extension ] : FTP_AUTOASCII;
    … …  
    137146                $temp = tmpfile();
    138147                if ( ! $this->ftp->fget($temp, $file) ) {
    139148                        fclose($temp);
    140                         return false;
     149                        return ''; //Blank document, File does exist, Its just blank.
    141150                }
    142151                fseek($temp, 0); //Skip back to the start of the file being written to
    143152                $contents = '';
    … …  
    287296                        return false;
    288297
    289298                $content = $this->get_contents($source);
    290                 if ( !$content )
     299                if ( false === $content )
    291300                        return false;
    292301
    293302                return $this->put_contents($destination,$content);