Changeset 28488

Timestamp:

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

Author:

wonderboymusic

Message:

Add access modifier (public) to methods in WP_Filesystem_Direct.

See #27881, #22234.

File:

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

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

@@ -22 +22 @@
      * @param mixed $arg ignored argument
      */
-    function __construct($arg) {
+    public function __construct($arg) {
         $this->method = 'direct';
         $this->errors = new WP_Error();
@@ -33 +33 @@
      * @return string|bool The function returns the read data or false on failure.
      */
-    function get_contents($file) {
+    public function get_contents($file) {
         return @file_get_contents($file);
     }
@@ -43 +43 @@
      * @return array|bool the file contents in an array or false on failure.
      */
-    function get_contents_array($file) {
+    public function get_contents_array($file) {
         return @file($file);
     }
@@ -55 +55 @@
      * @return bool False upon failure.
      */
-    function put_contents( $file, $contents, $mode = false ) {
+    public function put_contents( $file, $contents, $mode = false ) {
         $fp = @fopen( $file, 'wb' );
         if ( ! $fp )
@@ -83 +83 @@
      * @return string|bool the current working directory on success, or false on failure.
      */
-    function cwd() {
+    public function cwd() {
         return @getcwd();
     }
@@ -93 +93 @@
      * @return bool Returns true on success or false on failure.
      */
-    function chdir($dir) {
+    public function chdir($dir) {
         return @chdir($dir);
     }
@@ -105 +105 @@
      * @return bool Returns true on success or false on failure.
      */
-    function chgrp($file, $group, $recursive = false) {
+    public function chgrp($file, $group, $recursive = false) {
         if ( ! $this->exists($file) )
             return false;
@@ -129 +129 @@
      * @return bool Returns true on success or false on failure.
      */
-    function chmod($file, $mode = false, $recursive = false) {
+    public function chmod($file, $mode = false, $recursive = false) {
         if ( ! $mode ) {
             if ( $this->is_file($file) )
@@ -158 +158 @@
      * @return bool Returns true on success or false on failure.
      */
-    function chown($file, $owner, $recursive = false) {
+    public function chown($file, $owner, $recursive = false) {
         if ( ! $this->exists($file) )
             return false;
@@ -179 +179 @@
      * @return string|bool Username of the user or false on error.
      */
-    function owner($file) {
+    public function owner($file) {
         $owneruid = @fileowner($file);
         if ( ! $owneruid )
@@ -197 +197 @@
      * @return string Mode of the file (last 3 digits).
      */
-    function getchmod($file) {
+    public function getchmod($file) {
         return substr( decoct( @fileperms( $file ) ), -3 );
     }
 
-    function group($file) {
+    public function group($file) {
         $gid = @filegroup($file);
         if ( ! $gid )
@@ -211 +211 @@
     }
 
-    function copy($source, $destination, $overwrite = false, $mode = false) {
+    public function copy($source, $destination, $overwrite = false, $mode = false) {
         if ( ! $overwrite && $this->exists($destination) )
             return false;
@@ -221 +221 @@
     }
 
-    function move($source, $destination, $overwrite = false) {
+    public function move($source, $destination, $overwrite = false) {
         if ( ! $overwrite && $this->exists($destination) )
             return false;
@@ -237 +237 @@
     }
 
-    function delete($file, $recursive = false, $type = false) {
+    public function delete($file, $recursive = false, $type = false) {
         if ( empty( $file ) ) // Some filesystems report this as /, which can cause non-expected recursive deletion of all files in the filesystem.
             return false;
@@ -265 +265 @@
     }
 
-    function exists($file) {
+    public function exists($file) {
         return @file_exists($file);
     }
 
-    function is_file($file) {
+    public function is_file($file) {
         return @is_file($file);
     }
 
-    function is_dir($path) {
+    public function is_dir($path) {
         return @is_dir($path);
     }
 
-    function is_readable($file) {
+    public function is_readable($file) {
         return @is_readable($file);
     }
 
-    function is_writable($file) {
+    public function is_writable($file) {
         return @is_writable($file);
     }
 
-    function atime($file) {
+    public function atime($file) {
         return @fileatime($file);
     }
 
-    function mtime($file) {
+    public function mtime($file) {
         return @filemtime($file);
     }
 
-    function size($file) {
+    public function size($file) {
         return @filesize($file);
     }
 
-    function touch($file, $time = 0, $atime = 0) {
+    public function touch($file, $time = 0, $atime = 0) {
         if ($time == 0)
             $time = time();
@@ -305 +305 @@
     }
 
-    function mkdir($path, $chmod = false, $chown = false, $chgrp = false) {
+    public function mkdir($path, $chmod = false, $chown = false, $chgrp = false) {
         // safe mode fails with a trailing slash under certain PHP versions.
         $path = untrailingslashit($path);
@@ -324 +324 @@
     }
 
-    function rmdir($path, $recursive = false) {
+    public function rmdir($path, $recursive = false) {
         return $this->delete($path, $recursive);
     }
 
-    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);