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

Timestamp:

01/29/2020 12:43:23 AM (5 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-ftpsockets.php (modified) (6 diffs)

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

@@ -151 +151 @@
             reset_mbstring_encoding();
 
-            return ''; // Blank document, File does exist, It's just blank.
+            return ''; // Blank document. File does exist, it's just blank.
         }
 
         reset_mbstring_encoding();
 
-        fseek( $temphandle, 0 ); // Skip back to the start of the file being written to
+        fseek( $temphandle, 0 ); // Skip back to the start of the file being written to.
         $contents = '';
 
@@ -199 +199 @@
         }
 
-        // The FTP class uses string functions internally during file download/upload
+        // The FTP class uses string functions internally during file download/upload.
         mbstring_binary_safe_encoding();
 
@@ -212 +212 @@
         }
 
-        fseek( $temphandle, 0 ); // Skip back to the start of the file being written to
+        fseek( $temphandle, 0 ); // Skip back to the start of the file being written to.
 
         $ret = $this->ftp->fput( $file, $temphandle );
@@ -284 +284 @@
         }
 
-        // chmod the file or directory
+        // chmod the file or directory.
         return $this->ftp->chmod( $file, $mode );
     }
@@ -409 +409 @@
         }
 
-        return ! empty( $list ); //empty list = no file, so invert.
+        return ! empty( $list ); // Empty list = no file, so invert.
         // Return $this->ftp->is_exists($file); has issues with ABOR+426 responses on the ncFTPd server.
     }
@@ -636 +636 @@
             }
 
-            // Replace symlinks formatted as "source -> target" with just the source name
+            // Replace symlinks formatted as "source -> target" with just the source name.
             if ( $struc['islink'] ) {
                 $struc['name'] = preg_replace( '/(\s*->\s*.*)$/', '', $struc['name'] );
             }
 
-            // Add the Octal representation of the file permissions
+            // Add the octal representation of the file permissions.
             $struc['permsn'] = $this->getnumchmodfromh( $struc['perms'] );