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

Timestamp:

01/29/2020 12:43:23 AM (6 years ago)

Author:

SergeyBiryukov

Message:

Docs: Improve inline comments per the documentation standards.

Includes minor code layout fixes for better readability.

See #48303.

File:

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

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

@@ -39 +39 @@
         }
 
-        // 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' ) ) {
             define( 'FS_TIMEOUT', 240 );
@@ -114 +113 @@
         }
 
-        // 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 ) {
@@ -147 +146 @@
         }
 
-        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 = '';
 
@@ -204 +203 @@
         }
 
-        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.
 
         $ret = ftp_fput( $this->link, $file, $temp, FTP_BINARY );
@@ -274 +273 @@
         }
 
-        // chmod the file or directory
+        // chmod the file or directory.
         if ( ! function_exists( 'ftp_chmod' ) ) {
             return (bool) ftp_site( $this->link, sprintf( 'CHMOD %o %s', $mode, $file ) );
@@ -406 +405 @@
         }
 
-        return ! empty( $list ); //empty list = no file, so invert.
+        return ! empty( $list ); // Empty list = no file, so invert.
     }
 
@@ -574 +573 @@
                 $lucifer[3] += 2000;
             } else {
-                $lucifer[3] += 1900; // 4digit year fix
+                $lucifer[3] += 1900; // 4-digit year fix.
             }
             $b['isdir'] = ( $lucifer[7] == '<DIR>' );
@@ -594 +593 @@
             $lucifer = preg_split( '/[ ]/', $line, 9, PREG_SPLIT_NO_EMPTY );
             if ( $lucifer ) {
-                //echo $line."\n";
+                // echo $line."\n";
                 $lcount = count( $lucifer );
                 if ( $lcount < 8 ) {
@@ -638 +637 @@
         }
 
-        // Replace symlinks formatted as "source -> target" with just the source name
+        // Replace symlinks formatted as "source -> target" with just the source name.
         if ( isset( $b['islink'] ) && $b['islink'] ) {
             $b['name'] = preg_replace( '/(\s*->\s*.*)$/', '', $b['name'] );