Changeset 25305 for trunk/src/wp-admin/includes/class-wp-filesystem-ftpext.php

Timestamp:

09/09/2013 02:54:50 AM (13 years ago)

Author:

dd32

Message:

WP_Filesystem: Let the code breathe, add some additional whitespace between method definitions and comments.

File:

• trunk/src/wp-admin/includes/class-wp-filesystem-ftpext.php (modified) (12 diffs)

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

@@ -24 +24 @@
         $this->errors = new WP_Error();
 
-        //Check if possible to use ftp functions.
+        // Check if possible to use ftp functions.
         if ( ! extension_loaded('ftp') ) {
             $this->errors->add('no_ftp_ext', __('The ftp PHP extension is not available'));
@@ -30 +30 @@
         }
 
-        // Set defaults:
-        //This Class uses the timeout on a per-connection basis, Others use it on a per-action basis.
+        // This Class uses the timeout on a per-connection basis, Others use it on a per-action basis.
 
         if ( ! defined('FS_TIMEOUT') )
@@ -81 +80 @@
         }
 
-        //Set the Connection to use Passive FTP
+        // Set the Connection to use Passive FTP
         @ftp_pasv( $this->link, true );
         if ( @ftp_get_option($this->link, FTP_TIMEOUT_SEC) < FS_TIMEOUT )
@@ -99 +98 @@
             return false;
 
-        fseek($temp, 0); //Skip back to the start of the file being written to
+        fseek( $temp, 0 ); // Skip back to the start of the file being written to
         $contents = '';
 
@@ -109 +108 @@
         return $contents;
     }
+
     function get_contents_array($file) {
         return explode("\n", $this->get_contents($file));
@@ -137 +137 @@
         return $ret;
     }
+
     function cwd() {
         $cwd = @ftp_pwd($this->link);
@@ -143 +144 @@
         return $cwd;
     }
+
     function chdir($dir) {
         return @ftp_chdir($this->link, $dir);
     }
+
     function chgrp($file, $group, $recursive = false ) {
         return false;
     }
+
     function chmod($file, $mode = false, $recursive = false) {
         if ( ! $mode ) {
@@ -171 +175 @@
         return (bool)@ftp_chmod($this->link, $mode, $file);
     }
+
     function chown($file, $owner, $recursive = false ) {
         return false;
     }
+
     function owner($file) {
         $dir = $this->dirlist($file);
         return $dir[$file]['owner'];
     }
+
     function getchmod($file) {
         $dir = $this->dirlist($file);
         return $dir[$file]['permsn'];
     }
+
     function group($file) {
         $dir = $this->dirlist($file);
         return $dir[$file]['group'];
     }
+
     function copy($source, $destination, $overwrite = false, $mode = false) {
         if ( ! $overwrite && $this->exists($destination) )
@@ -194 +203 @@
         return $this->put_contents($destination, $content, $mode);
     }
+
     function move($source, $destination, $overwrite = false) {
         return ftp_rename($this->link, $source, $destination);
@@ -217 +227 @@
         return !empty($list); //empty list = no file, so invert.
     }
+
     function is_file($file) {
         return $this->exists($file) && !$this->is_dir($file);
     }
+
     function is_dir($path) {
         $cwd = $this->cwd();
@@ -229 +241 @@
         return false;
     }
+
     function is_readable($file) {
-        //Get dir list, Check if the file is readable by the current user??
         return true;
     }
+
     function is_writable($file) {
-        //Get dir list, Check if the file is writable by the current user??
         return true;
     }
+
     function atime($file) {
         return false;
     }
+
     function mtime($file) {
         return ftp_mdtm($this->link, $file);
     }
+
     function size($file) {
         return ftp_size($this->link, $file);
     }
+
     function touch($file, $time = 0, $atime = 0) {
         return false;
     }
+
     function mkdir($path, $chmod = false, $chown = false, $chgrp = false) {
         $path = untrailingslashit($path);
@@ -263 +280 @@
         return true;
     }
+
     function rmdir($path, $recursive = false) {
         return $this->delete($path, $recursive);