WordPress.org
Get WordPress

Make WordPress Core

Changeset 10919


Ignore:
Timestamp:
04/13/2009 04:11:02 PM (17 years ago)
Author:
ryan
Message:

WPFS cleanups. Props DD32. fixes #9525

Location:
trunk/wp-admin/includes
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-admin/includes/class-wp-filesystem-base.php

    r10150 r10919  
    4747         */
    4848        function abspath() {
    49                 if ( defined('FTP_BASE') && strpos($this->method, 'ftp') !== false )
    50                         return FTP_BASE;
    5149                $folder = $this->find_folder(ABSPATH);
    5250                //Perhaps the FTP folder is rooted at the WordPress install, Check for wp-includes folder in root, Could have some false positives, but rare.
    … …  
    6361         */
    6462        function wp_content_dir() {
    65                 if ( defined('FTP_CONTENT_DIR') && strpos($this->method, 'ftp') !== false )
    66                         return FTP_CONTENT_DIR;
    6763                return $this->find_folder(WP_CONTENT_DIR);
    6864        }
    … …  
    7672         */
    7773        function wp_plugins_dir() {
    78                 if ( defined('FTP_PLUGIN_DIR') && strpos($this->method, 'ftp') !== false )
    79                         return FTP_PLUGIN_DIR;
    8074                return $this->find_folder(WP_PLUGIN_DIR);
    8175        }
    … …  
    143137        function find_folder($folder) {
    144138
     139                if ( strpos($this->method, 'ftp') !== false ) {
     140                        $constant_overrides = array( 'FTP_BASE' => ABSPATH, 'FTP_CONTENT_DIR' => WP_CONTENT_DIR, 'FTP_PLUGIN_DIR' => WP_PLUGIN_DIR );
     141                        foreach ( $constant_overrides as $constant => $dir )
     142                                if ( defined($constant) && $folder === $dir )
     143                                        return trailingslashit(constant($constant));
     144                } elseif ( 'direct' == $this->method ) {
     145                        return trailingslashit($folder);
     146                }
     147
    145148                $folder = preg_replace('|^([a-z]{1}):|i', '', $folder); //Strip out windows driveletter if its there.
    146149                $folder = str_replace('\\', '/', $folder); //Windows path sanitiation
    … …  
    150153
    151154                if ( $this->exists($folder) ) { //Folder exists at that absolute path.
     155                        $folder = trailingslashit($folder);
    152156                        $this->cache[ $folder ] = $folder;
    153157                        return $folder;
    … …  
    190194                        // If it cant find WordPress down that route, it'll continue onto the next folder level, and see if that matches, and so on.
    191195                        // If it reaches the end, and still cant find it, it'll return false for the entire function.
    192                         if( isset($files[ $key ]) ){
     196                        if ( isset($files[ $key ]) ){
    193197                                //Lets try that folder:
    194198                                $newdir = trailingslashit(path_join($base, $key));
    195                                 if( $this->verbose )
     199                                if ( $this->verbose )
    196200                                        printf( __('Changing to %s') . '<br/>', $newdir );
    197                                 if( $ret = $this->search_for_folder( $folder, $newdir, $loop) )
     201                                if ( $ret = $this->search_for_folder( $folder, $newdir, $loop) )
    198202                                        return $ret;
    199203                        }
    … …  
    201205
    202206                //Only check this as a last resort, to prevent locating the incorrect install. All above proceeedures will fail quickly if this is the right branch to take.
    203                 if(isset( $files[ $last_path ] ) ) {
    204                         if( $this->verbose )
     207                if (isset( $files[ $last_path ] ) ) {
     208                        if ( $this->verbose )
    205209                                printf( __('Found %s') . '<br/>',  $base . $last_path );
    206                         return $base . $last_path;
     210                        return trailingslashit($base . $last_path);
    207211                }
    208                 if( $loop )
     212                if ( $loop )
    209213                        return false;//Prevent tihs function looping again.
    210214                //As an extra last resort, Change back to / if the folder wasnt found. This comes into effect when the CWD is /home/user/ but WP is at /var/www/.... mainly dedicated setups.

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

    r10051 r10919  
    1515 * @uses WP_Filesystem_Base Extends class
    1616 */
    17 class WP_Filesystem_Direct  extends WP_Filesystem_Base {
     17class WP_Filesystem_Direct extends WP_Filesystem_Base {
    1818        var $permission = null;
    19         var $errors = array();
     19        var $errors = null;
    2020        function WP_Filesystem_Direct($arg) {
    2121                $this->method = 'direct';
    … …  
    5050        }
    5151        function chgrp($file, $group, $recursive = false) {
    52                 if( ! $this->exists($file) )
    53                         return false;
    54                 if( ! $recursive )
     52                if ( ! $this->exists($file) )
     53                        return false;
     54                if ( ! $recursive )
    5555                        return @chgrp($file, $group);
    56                 if( ! $this->is_dir($file) )
     56                if ( ! $this->is_dir($file) )
    5757                        return @chgrp($file, $group);
    5858                //Is a directory, and we want recursive
    5959                $file = trailingslashit($file);
    6060                $filelist = $this->dirlist($file);
    61                 foreach($filelist as $filename)
     61                foreach ($filelist as $filename)
    6262                        $this->chgrp($file . $filename, $group, $recursive);
    6363
    … …  
    6565        }
    6666        function chmod($file, $mode = false, $recursive = false) {
    67                 if( ! $mode )
     67                if ( ! $mode )
    6868                        $mode = $this->permission;
    69                 if( ! $this->exists($file) )
    70                         return false;
    71                 if( ! $recursive )
     69                if ( ! $this->exists($file) )
     70                        return false;
     71                if ( ! $recursive )
    7272                        return @chmod($file,$mode);
    73                 if( ! $this->is_dir($file) )
     73                if ( ! $this->is_dir($file) )
    7474                        return @chmod($file, $mode);
    7575                //Is a directory, and we want recursive
    7676                $file = trailingslashit($file);
    7777                $filelist = $this->dirlist($file);
    78                 foreach($filelist as $filename)
     78                foreach ($filelist as $filename)
    7979                        $this->chmod($file . $filename, $mode, $recursive);
    8080
    … …  
    8282        }
    8383        function chown($file, $owner, $recursive = false) {
    84                 if( ! $this->exists($file) )
    85                         return false;
    86                 if( ! $recursive )
     84                if ( ! $this->exists($file) )
     85                        return false;
     86                if ( ! $recursive )
    8787                        return @chown($file, $owner);
    88                 if( ! $this->is_dir($file) )
     88                if ( ! $this->is_dir($file) )
    8989                        return @chown($file, $owner);
    9090                //Is a directory, and we want recursive
    9191                $filelist = $this->dirlist($file);
    92                 foreach($filelist as $filename){
     92                foreach ($filelist as $filename){
    9393                        $this->chown($file . '/' . $filename, $owner, $recursive);
    9494                }
    … …  
    9797        function owner($file) {
    9898                $owneruid = @fileowner($file);
    99                 if( ! $owneruid )
    100                         return false;
    101                 if( ! function_exists('posix_getpwuid') )
     99                if ( ! $owneruid )
     100                        return false;
     101                if ( ! function_exists('posix_getpwuid') )
    102102                        return $owneruid;
    103103                $ownerarray = posix_getpwuid($owneruid);
    … …  
    105105        }
    106106        function getchmod($file) {
    107                 return @fileperms($file);
     107                return substr(decoct(@fileperms($file)),3);
    108108        }
    109109        function group($file) {
    110110                $gid = @filegroup($file);
    111                 if( ! $gid )
    112                         return false;
    113                 if( ! function_exists('posix_getgrgid') )
     111                if ( ! $gid )
     112                        return false;
     113                if ( ! function_exists('posix_getgrgid') )
    114114                        return $gid;
    115115                $grouparray = posix_getgrgid($gid);
    … …  
    118118
    119119        function copy($source, $destination, $overwrite = false) {
    120                 if( ! $overwrite && $this->exists($destination) )
     120                if ( ! $overwrite && $this->exists($destination) )
    121121                        return false;
    122122                return copy($source, $destination);
    … …  
    125125        function move($source, $destination, $overwrite = false) {
    126126                //Possible to use rename()?
    127                 if( $this->copy($source, $destination, $overwrite) && $this->exists($destination) ){
     127                if ( $this->copy($source, $destination, $overwrite) && $this->exists($destination) ){
    128128                        $this->delete($source);
    129129                        return true;
    … …  
    134134
    135135        function delete($file, $recursive = false) {
     136                if ( empty($file) ) //Some filesystems report this as /, which can cause non-expected recursive deletion of all files in the filesystem.
     137                        return false;
    136138                $file = str_replace('\\', '/', $file); //for win32, occasional problems deleteing files otherwise
    137139
    138                 if( $this->is_file($file) )
     140                if ( $this->is_file($file) )
    139141                        return @unlink($file);
    140                 if( ! $recursive && $this->is_dir($file) )
     142                if ( ! $recursive && $this->is_dir($file) )
    141143                        return @rmdir($file);
    142144
    … …  
    146148
    147149                $retval = true;
    148                 if( is_array($filelist) ) //false if no files, So check first.
    149                         foreach($filelist as $filename => $fileinfo)
    150                                 if( ! $this->delete($file . $filename, $recursive) )
     150                if ( is_array($filelist) ) //false if no files, So check first.
     151                        foreach ($filelist as $filename => $fileinfo)
     152                                if ( ! $this->delete($file . $filename, $recursive) )
    151153                                        $retval = false;
    152154
    153                 if( ! @rmdir($file) )
    154                         return false;
     155                if ( file_exists($file) && ! @rmdir($file) )
     156                        $retval = false;
    155157                return $retval;
    156158        }
    … …  
    188190
    189191        function touch($file, $time = 0, $atime = 0){
    190                 if($time == 0)
     192                if ($time == 0)
    191193                        $time = time();
    192                 if($atime == 0)
     194                if ($atime == 0)
    193195                        $atime = time();
    194196                return @touch($file, $time, $atime);
    … …  
    196198
    197199        function mkdir($path, $chmod = false, $chown = false, $chgrp = false){
    198                 if( ! $chmod)
     200                if ( ! $chmod)
    199201                        $chmod = $this->permission;
    200202
    201                 if( ! @mkdir($path, $chmod) )
    202                         return false;
    203                 if( $chown )
     203                if ( ! @mkdir($path, $chmod) )
     204                        return false;
     205                if ( $chown )
    204206                        $this->chown($path, $chown);
    205                 if( $chgrp )
     207                if ( $chgrp )
    206208                        $this->chgrp($path, $chgrp);
    207209                return true;
    … …  
    210212        function rmdir($path, $recursive = false) {
    211213                //Currently unused and untested, Use delete() instead.
    212                 if( ! $recursive )
     214                if ( ! $recursive )
    213215                        return @rmdir($path);
    214216                //recursive:
    215217                $filelist = $this->dirlist($path);
    216                 foreach($filelist as $filename => $det) {
     218                foreach ($filelist as $filename => $det) {
    217219                        if ( '/' == substr($filename, -1, 1) )
    218220                                $this->rmdir($path . '/' . $filename, $recursive);
    … …  
    223225
    224226        function dirlist($path, $incdot = false, $recursive = false) {
    225                 if( $this->is_file($path) ) {
     227                if ( $this->is_file($path) ) {
    226228                        $limitFile = basename($path);
    227229                        $path = dirname($path);
    … …  
    229231                        $limitFile = false;
    230232                }
    231                 if( ! $this->is_dir($path) )
     233                if ( ! $this->is_dir($path) )
    232234                        return false;
    233235
    … …  
    240242                        $struc['name'] = $entry;
    241243
    242                         if( '.' == $struc['name'] || '..' == $struc['name'] )
     244                        if ( '.' == $struc['name'] || '..' == $struc['name'] )
    243245                                continue; //Do not care about these folders.
    244                         if( '.' == $struc['name'][0] && !$incdot)
     246                        if ( '.' == $struc['name'][0] && !$incdot)
    245247                                continue;
    246                         if( $limitFile && $struc['name'] != $limitFile)
     248                        if ( $limitFile && $struc['name'] != $limitFile)
    247249                                continue;
    248250
    … …  
    259261
    260262                        if ( 'd' == $struc['type'] ) {
    261                                 if( $recursive )
     263                                if ( $recursive )
    262264                                        $struc['files'] = $this->dirlist($path . '/' . $struc['name'], $incdot, $recursive);
    263265                                else

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

    r10150 r10919  
    170170        function getchmod($file) {
    171171                $dir = $this->dirlist($file);
    172                 return $dir[$file]['permsn'];
     172                return $this->getnumchmodfromh( $dir[basename($file)]['perms'] );
    173173        }
    174174        function group($file) {
    … …  
    188188        }
    189189
    190         function delete($file,$recursive=false) {
     190        function delete($file ,$recursive = false ) {
     191                if ( empty($file) )
     192                        return false;
    191193                if ( $this->is_file($file) )
    192194                        return @ftp_delete($this->link, $file);
    … …  
    322324
    323325        function dirlist($path = '.', $incdot = false, $recursive = false) {
    324                 if( $this->is_file($path) ) {
    325                         $limitFile = basename($path);
    326                         $path = dirname($path) . '/';
     326
     327                if ( substr($path, -1) !== '/') {
     328                        $limit = basename($path);
     329                        $path = trailingslashit(dirname($path));
    327330                } else {
    328                         $limitFile = false;
     331                        $limit = false;
    329332                }
    330333
    … …  
    340343                                continue;
    341344
    342                         if ( '.' == $entry["name"] || '..' == $entry["name"] )
     345                        if ( '.' == $entry['name'] || '..' == $entry['name'] )
     346                                continue;
     347
     348                        if ( $limit && $entry['name'] != $limit )
    343349                                continue;
    344350

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

    r8990 r10919  
    187187        function getchmod($file) {
    188188                $dir = $this->dirlist($file);
    189                 return $dir[$file]['permsn'];
     189                return $this->getnumchmodfromh( $dir[basename($file)]['perms'] );
    190190        }
    191191
    … …  
    282282
    283283        function dirlist($path = '.', $incdot = false, $recursive = false ) {
    284                 if( $this->is_file($path) ) {
    285                         $limitFile = basename($path);
    286                         $path = dirname($path) . '/';
     284
     285                if ( substr($path, -1) !== '/') {
     286                        $limit = basename($path);
     287                        $path = trailingslashit(dirname($path));
    287288                } else {
    288                         $limitFile = false;
     289                        $limit = false;
    289290                }
    290291
    … …  
    292293                if( ! $list )
    293294                        return false;
     295
    294296                if( empty($list) )
    295297                        return array();
    … …  
    297299                $ret = array();
    298300                foreach ( $list as $struc ) {
     301                        if ( $limit && $struc['name'] != $limit )
     302                                continue;
    299303
    300304                        if ( 'd' == $struc['type'] ) {
Note: See TracChangeset for help on using the changeset viewer.