WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
06/19/2020 10:34:26 AM (6 years ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Add some space around control structures in WP_Filesystem_* classes for consistency and better readability.

Additionally, synchronize $tempfile and $temphandle variable names in WP_Filesystem_FTPext and WP_Filesystem_ftpsockets.

See #49542.

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r47808 r48089  
    3737                        return;
    3838                }
     39
    3940                $this->ftp = new ftp();
    4041
    … …  
    8889                                )
    8990                        );
     91
    9092                        return false;
    9193                }
    … …  
    100102                                )
    101103                        );
     104
    102105                        return false;
    103106                }
    … …  
    112115                                )
    113116                        );
     117
    114118                        return false;
    115119                }
    … …  
    118122                $this->ftp->Passive( true );
    119123                $this->ftp->setTimeout( FS_TIMEOUT );
     124
    120125                return true;
    121126        }
    … …  
    135140                }
    136141
    137                 $temp = wp_tempnam( $file );
    138 
    139                 $temphandle = fopen( $temp, 'w+' );
     142                $tempfile  = wp_tempnam( $file );
     143                $temphandle = fopen( $tempfile, 'w+' );
     144
    140145                if ( ! $temphandle ) {
    141                         unlink( $temp );
     146                        unlink( $tempfile );
    142147                        return false;
    143148                }
    … …  
    147152                if ( ! $this->ftp->fget( $temphandle, $file ) ) {
    148153                        fclose( $temphandle );
    149                         unlink( $temp );
     154                        unlink( $tempfile );
    150155
    151156                        reset_mbstring_encoding();
    … …  
    164169
    165170                fclose( $temphandle );
    166                 unlink( $temp );
     171                unlink( $tempfile );
     172
    167173                return $contents;
    168174        }
    … …  
    192198         */
    193199        public function put_contents( $file, $contents, $mode = false ) {
    194                 $temp       = wp_tempnam( $file );
    195                 $temphandle = @fopen( $temp, 'w+' );
     200                $tempfile   = wp_tempnam( $file );
     201                $temphandle = @fopen( $tempfile, 'w+' );
     202
    196203                if ( ! $temphandle ) {
    197                         unlink( $temp );
     204                        unlink( $tempfile );
    198205                        return false;
    199206                }
    … …  
    203210
    204211                $bytes_written = fwrite( $temphandle, $contents );
     212
    205213                if ( false === $bytes_written || strlen( $contents ) != $bytes_written ) {
    206214                        fclose( $temphandle );
    207                         unlink( $temp );
     215                        unlink( $tempfile );
    208216
    209217                        reset_mbstring_encoding();
    … …  
    219227
    220228                fclose( $temphandle );
    221                 unlink( $temp );
     229                unlink( $tempfile );
    222230
    223231                $this->chmod( $file, $mode );
    … …  
    235243        public function cwd() {
    236244                $cwd = $this->ftp->pwd();
     245
    237246                if ( $cwd ) {
    238247                        $cwd = trailingslashit( $cwd );
    239248                }
     249
    240250                return $cwd;
    241251        }
    … …  
    279289                if ( $recursive && $this->is_dir( $file ) ) {
    280290                        $filelist = $this->dirlist( $file );
     291
    281292                        foreach ( (array) $filelist as $filename => $filemeta ) {
    282293                                $this->chmod( $file . '/' . $filename, $mode, $recursive );
    … …  
    298309        public function owner( $file ) {
    299310                $dir = $this->dirlist( $file );
     311
    300312                return $dir[ $file ]['owner'];
    301313        }
    … …  
    311323        public function getchmod( $file ) {
    312324                $dir = $this->dirlist( $file );
     325
    313326                return $dir[ $file ]['permsn'];
    314327        }
    … …  
    324337        public function group( $file ) {
    325338                $dir = $this->dirlist( $file );
     339
    326340                return $dir[ $file ]['group'];
    327341        }
    … …  
    346360
    347361                $content = $this->get_contents( $source );
     362
    348363                if ( false === $content ) {
    349364                        return false;
    … …  
    384399                        return false;
    385400                }
     401
    386402                if ( 'f' === $type || $this->is_file( $file ) ) {
    387403                        return $this->ftp->delete( $file );
    388404                }
     405
    389406                if ( ! $recursive ) {
    390407                        return $this->ftp->rmdir( $file );
    … …  
    425442                        return false;
    426443                }
     444
    427445                if ( $this->exists( $file ) ) {
    428446                        return true;
    429447                }
     448
    430449                return false;
    431450        }
    … …  
    441460        public function is_dir( $path ) {
    442461                $cwd = $this->cwd();
     462
    443463                if ( $this->chdir( $path ) ) {
    444464                        $this->chdir( $cwd );
    445465                        return true;
    446466                }
     467
    447468                return false;
    448469        }
    … …  
    542563        public function mkdir( $path, $chmod = false, $chown = false, $chgrp = false ) {
    543564                $path = untrailingslashit( $path );
     565
    544566                if ( empty( $path ) ) {
    545567                        return false;
    … …  
    549571                        return false;
    550572                }
     573
    551574                if ( ! $chmod ) {
    552575                        $chmod = FS_CHMOD_DIR;
    553576                }
     577
    554578                $this->chmod( $path, $chmod );
     579
    555580                return true;
    556581        }
    … …  
    606631
    607632                $list = $this->ftp->dirlist( $path );
     633
    608634                if ( empty( $list ) && ! $this->exists( $path ) ) {
    609635
    … …  
    614640
    615641                $ret = array();
     642
    616643                foreach ( $list as $struc ) {
    617644
Note: See TracChangeset for help on using the changeset viewer.