From 9a16e6230b15d4b020a60ef653cfab7a5c3ebd3c Mon Sep 17 00:00:00 2001 From: Scott Taylor Date: Fri, 16 Jan 2015 21:06:29 +0000 Subject: [PATCH] Calling `->chown()` and `->chgrp()` in `WP_Filesystem_FTPext` produces no side-effects. Remove the calls. `->chgrp()` is defined in the super class, and does nothing, can be removed from the child class. See #30799. git-svn-id: https://develop.svn.wordpress.org/trunk@31215 602fd350-edb4-49c9-b593-d223f7449a82 --- .../includes/class-wp-filesystem-ftpext.php | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/src/wp-admin/includes/class-wp-filesystem-ftpext.php b/src/wp-admin/includes/class-wp-filesystem-ftpext.php index b4809d4b0d..109acc16ff 100644 --- a/src/wp-admin/includes/class-wp-filesystem-ftpext.php +++ b/src/wp-admin/includes/class-wp-filesystem-ftpext.php @@ -170,15 +170,6 @@ class WP_Filesystem_FTPext extends WP_Filesystem_Base { return @ftp_chdir($this->link, $dir); } - /** - * @param string $file - * @param bool $group - * @param bool $recursive - */ - public function chgrp($file, $group, $recursive = false ) { - return false; - } - /** * @param string $file * @param int $mode @@ -365,10 +356,6 @@ class WP_Filesystem_FTPext extends WP_Filesystem_Base { if ( !@ftp_mkdir($this->link, $path) ) return false; $this->chmod($path, $chmod); - if ( $chown ) - $this->chown($path, $chown); - if ( $chgrp ) - $this->chgrp($path, $chgrp); return true; }