Ticket #36308: 36308.3.diff

src/wp-includes/functions.php

@@ -1729 +1729 @@
                 return true;
         }
 
+        // Windows normalized paths for local filesystem and network shares (forward slahes)
+        if ( preg_match( '#(^[a-zA-Z]+:/|^//[\w!@\#\$%\^\(\)\-\'{}\.~]{1,15})#', $path ) ) {
+                return true;
+        }
+
         // A path starting with / or \ is absolute; anything else is relative.
         return ( $path[0] == '/' || $path[0] == '\\' );
 }

src/wp-includes/post.php

@@ -320 +320 @@
 function get_attached_file( $attachment_id, $unfiltered = false ) {
         $file = get_post_meta( $attachment_id, '_wp_attached_file', true );
 
-        // If the file is relative, prepend upload dir.
-        if ( $file && 0 !== strpos( $file, '/' ) && ! preg_match( '|^.:\\\|', $file ) && ( ( $uploads = wp_get_upload_dir() ) && false === $uploads['error'] ) ) {
-                $file = $uploads['basedir'] . "/$file";
+        if ( ( $uploads = wp_get_upload_dir() ) && false === $uploads['error'] ) {
+                $file = path_join($uploads['basedir'], $file);
         }
 
         if ( $unfiltered ) {

tests/phpunit/tests/post.php

@@ -1354 +1354 @@
                 $this->assertEquals( $changeset_data, json_decode( get_post( $post_id )->post_content, true ) );
         }
 
+        /**
+         * @ticket 36308
+         */
+        function test_wp_get_attached_file() {
+        
+                $post = self::factory()->post->create_and_get( 
+                        array(
+                        'post_title' => 'example-page',
+                        'post_type' => 'post'
+                        )
+                );
+                
+                // Windows local file system path
+                
+                        $attachment = self::factory()->attachment->create_and_get( 
+                                array(
+                                'post_parent' => $post->ID,
+                                'file' => 'C:/WWW/Sites/demo/htdocs/wordpress/wp-content/uploads/2016/03/example.jpg',
+                                )
+                        );
+
+                        $attachment_path = get_attached_file( $attachment->ID );
+
+                        $this->assertEquals( $attachment_path, 'C:/WWW/Sites/demo/htdocs/wordpress/wp-content/uploads/2016/03/example.jpg' );
+
+                // Windows network shares path
+
+                        $attachment = self::factory()->attachment->create_and_get( 
+                                array(
+                                'post_parent' => $post->ID,
+                                'file' => '//ComputerName/ShareName/SubfolderName/example.txt',
+                                )
+                        );
+
+                        $attachment_path = get_attached_file( $attachment->ID );
+
+                        $this->assertEquals( $attachment_path, '//ComputerName/ShareName/SubfolderName/example.txt' );          
+        }
+
 }