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

Timestamp:

08/09/2022 11:32:01 AM (2 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Rename the $file parameter to $path in some WP_Filesystem_* methods.

This aims to bring more clarity to the code, and applies to methods where the path can be a file or a directory:

• WP_Filesystem_*::exists()
• WP_Filesystem_*::is_writable()

Follow-up to [6779], [25560].

See #55647.

File:

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

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

@@ -416 +416 @@
      *              and ftp_rawlist() to check for file existence.
      *
-     * @param string $file Path to file or directory.
-     * @return bool Whether $file exists or not.
-     */
-    public function exists( $file ) {
-        if ( $this->is_dir( $file ) ) {
+     * @param string $path Path to file or directory.
+     * @return bool Whether $path exists or not.
+     */
+    public function exists( $path ) {
+        if ( $this->is_dir( $path ) ) {
             return true;
         }
 
-        return ! empty( ftp_rawlist( $this->link, $file ) );
+        return ! empty( ftp_rawlist( $this->link, $path ) );
     }
 
@@ -476 +476 @@
      * @since 2.5.0
      *
-     * @param string $file Path to file or directory.
-     * @return bool Whether $file is writable.
-     */
-    public function is_writable( $file ) {
+     * @param string $path Path to file or directory.
+     * @return bool Whether $path is writable.
+     */
+    public function is_writable( $path ) {
         return true;
     }