Filesystem API: Resolve FIXME comment for WP_Filesystem_Direct::getchmod() by explicitly returning '0' in error case.

Developed in #10637

Follow-up to [11831].

Props vietcgi, westonruter.
See #10304.
Fixes #64426.


git-svn-id: https://develop.svn.wordpress.org/trunk@61398 602fd350-edb4-49c9-b593-d223f7449a82

Author: westonruter

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

@@ -249,15 +249,18 @@ public function owner( $file ) {
 	/**
 	 * Gets the permissions of the specified file or filepath in their octal format.
 	 *
-	 * FIXME does not handle errors in fileperms()
-	 *
 	 * @since 2.5.0
 	 *
 	 * @param string $file Path to the file.
-	 * @return string Mode of the file (the last 3 digits).
+	 * @return string Mode of the file (the last 3 digits), or the string "0" on failure.
 	 */
 	public function getchmod( $file ) {
-		return substr( decoct( @fileperms( $file ) ), -3 );
+		$perms = @fileperms( $file );
+		if ( false === $perms ) {
+			return '0';
+		}
+
+		return substr( decoct( $perms ), -3 );
 	}
 
 	/**

tests/phpunit/tests/filesystem/wpFilesystemDirect/getchmod.php

@@ -33,16 +33,17 @@ public function test_should_get_chmod_for_a_path_that_exists( $path ) {
 
 	/**
 	 * Tests that `WP_Filesystem_Direct::getchmod()` returns
-	 * the permissions for a path that does not exist.
+	 * "0" for a path that does not exist.
 	 *
 	 * @dataProvider data_paths_that_do_not_exist
 	 *
 	 * @ticket 57774
+	 * @ticket 64426
 	 *
 	 * @param string $path The path.
 	 */
-	public function test_should_get_chmod_for_a_path_that_does_not_exist( $path ) {
+	public function test_should_return_zero_for_a_path_that_does_not_exist( $path ) {
 		$actual = self::$filesystem->getchmod( self::$file_structure['test_dir']['path'] . $path );
-		$this->assertNotSame( '', $actual );
+		$this->assertSame( '0', $actual );
 	}
 }