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);