Changeset 28489

Timestamp:

05/19/2014 12:11:21 AM (11 years ago)

Author:

wonderboymusic

Message:

Add access modifier (public) to members and methods in WP_Filesystem_FTPext.

See #27881, #22234.

File:

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

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

@@ -16 +16 @@
  */
 class WP_Filesystem_FTPext extends WP_Filesystem_Base {
-    var $link;
-    var $errors = null;
-    var $options = array();
-
-    function __construct($opt='') {
+    public $link;
+    public $errors = null;
+    public $options = array();
+
+    public function __construct($opt='') {
         $this->method = 'ftpext';
         $this->errors = new WP_Error();
@@ -64 +64 @@
     }
 
-    function connect() {
+    public function connect() {
         if ( isset($this->options['ssl']) && $this->options['ssl'] && function_exists('ftp_ssl_connect') )
             $this->link = @ftp_ssl_connect($this->options['hostname'], $this->options['port'], FS_CONNECT_TIMEOUT);
@@ -88 +88 @@
     }
 
-    function get_contents( $file ) {
+    public function get_contents( $file ) {
         $tempfile = wp_tempnam($file);
         $temp = fopen($tempfile, 'w+');
@@ -109 +109 @@
     }
 
-    function get_contents_array($file) {
+    public function get_contents_array($file) {
         return explode("\n", $this->get_contents($file));
     }
 
-    function put_contents($file, $contents, $mode = false ) {
+    public function put_contents($file, $contents, $mode = false ) {
         $tempfile = wp_tempnam($file);
         $temp = fopen( $tempfile, 'wb+' );
@@ -144 +144 @@
     }
 
-    function cwd() {
+    public function cwd() {
         $cwd = @ftp_pwd($this->link);
         if ( $cwd )
@@ -151 +151 @@
     }
 
-    function chdir($dir) {
+    public function chdir($dir) {
         return @ftp_chdir($this->link, $dir);
     }
 
-    function chgrp($file, $group, $recursive = false ) {
+    public function chgrp($file, $group, $recursive = false ) {
         return false;
     }
 
-    function chmod($file, $mode = false, $recursive = false) {
+    public function chmod($file, $mode = false, $recursive = false) {
         if ( ! $mode ) {
             if ( $this->is_file($file) )
@@ -182 +182 @@
     }
 
-    function owner($file) {
+    public function owner($file) {
         $dir = $this->dirlist($file);
         return $dir[$file]['owner'];
     }
 
-    function getchmod($file) {
+    public function getchmod($file) {
         $dir = $this->dirlist($file);
         return $dir[$file]['permsn'];
     }
 
-    function group($file) {
+    public function group($file) {
         $dir = $this->dirlist($file);
         return $dir[$file]['group'];
     }
 
-    function copy($source, $destination, $overwrite = false, $mode = false) {
+    public function copy($source, $destination, $overwrite = false, $mode = false) {
         if ( ! $overwrite && $this->exists($destination) )
             return false;
@@ -206 +206 @@
     }
 
-    function move($source, $destination, $overwrite = false) {
+    public function move($source, $destination, $overwrite = false) {
         return ftp_rename($this->link, $source, $destination);
     }
 
-    function delete($file, $recursive = false, $type = false) {
+    public function delete($file, $recursive = false, $type = false) {
         if ( empty($file) )
             return false;
@@ -225 +225 @@
     }
 
-    function exists($file) {
+    public function exists($file) {
         $list = @ftp_nlist($this->link, $file);
         return !empty($list); //empty list = no file, so invert.
     }
 
-    function is_file($file) {
+    public function is_file($file) {
         return $this->exists($file) && !$this->is_dir($file);
     }
 
-    function is_dir($path) {
+    public function is_dir($path) {
         $cwd = $this->cwd();
         $result = @ftp_chdir($this->link, trailingslashit($path) );
@@ -244 +244 @@
     }
 
-    function is_readable($file) {
+    public function is_readable($file) {
         return true;
     }
 
-    function is_writable($file) {
+    public function is_writable($file) {
         return true;
     }
 
-    function atime($file) {
+    public function atime($file) {
         return false;
     }
 
-    function mtime($file) {
+    public function mtime($file) {
         return ftp_mdtm($this->link, $file);
     }
 
-    function size($file) {
+    public function size($file) {
         return ftp_size($this->link, $file);
     }
 
-    function touch($file, $time = 0, $atime = 0) {
+    public function touch($file, $time = 0, $atime = 0) {
         return false;
     }
 
-    function mkdir($path, $chmod = false, $chown = false, $chgrp = false) {
+    public function mkdir($path, $chmod = false, $chown = false, $chgrp = false) {
         $path = untrailingslashit($path);
         if ( empty($path) )
@@ -283 +283 @@
     }
 
-    function rmdir($path, $recursive = false) {
+    public function rmdir($path, $recursive = false) {
         return $this->delete($path, $recursive);
     }
 
-    function parselisting($line) {
+    public function parselisting($line) {
         static $is_windows;
         if ( is_null($is_windows) )
@@ -360 +360 @@
     }
 
-    function dirlist($path = '.', $include_hidden = true, $recursive = false) {
+    public function dirlist($path = '.', $include_hidden = true, $recursive = false) {
         if ( $this->is_file($path) ) {
             $limit_file = basename($path);
@@ -409 +409 @@
     }
 
-    function __destruct() {
+    public function __destruct() {
         if ( $this->link )
             ftp_close($this->link);