fixed #15118 [Filesystem] mirroring a symlink copies absolute file path

danepowell · fabpot · commit a83d525b902b · 2015-10-06T17:23:28.000+02:00
diff --git a/src/Symfony/Component/Filesystem/Filesystem.php b/src/Symfony/Component/Filesystem/Filesystem.php
@@ -402,7 +402,7 @@ public function mirror($originDir, $targetDir, \Traversable $iterator = null, $o
                 }
             } else {
                 if (is_link($file)) {
-                    $this->symlink($file->getRealPath(), $target);
+                    $this->symlink($file->getLinkTarget(), $target);
                 } elseif (is_dir($file)) {
                     $this->mkdir($target);
                 } elseif (is_file($file)) {
diff --git a/src/Symfony/Component/Filesystem/Tests/FilesystemTest.php b/src/Symfony/Component/Filesystem/Tests/FilesystemTest.php
@@ -906,7 +906,7 @@ public function testMirrorCopiesRelativeLinkedContents()
         $this->assertTrue(is_dir($targetPath));
         $this->assertFileEquals($sourcePath.'/nested/file1.txt', $targetPath.DIRECTORY_SEPARATOR.'link1/file1.txt');
         $this->assertTrue(is_link($targetPath.DIRECTORY_SEPARATOR.'link1'));
-        $this->assertEquals($sourcePath.'nested', readlink($targetPath.DIRECTORY_SEPARATOR.'link1'));
+        $this->assertEquals('nested', readlink($targetPath.DIRECTORY_SEPARATOR.'link1'));
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -402,7 +402,7 @@ public function mirror($originDir, $targetDir, \Traversable $iterator = null, $o`
`402`	`402`	`}`
`403`	`403`	`} else {`
`404`	`404`	`if (is_link($file)) {`
`405`		`- $this->symlink($file->getRealPath(), $target);`
	`405`	`+ $this->symlink($file->getLinkTarget(), $target);`
`406`	`406`	`} elseif (is_dir($file)) {`
`407`	`407`	`$this->mkdir($target);`
`408`	`408`	`} elseif (is_file($file)) {`
Original file line number	Diff line number	Diff line change
`@@ -906,7 +906,7 @@ public function testMirrorCopiesRelativeLinkedContents()`
`906`	`906`	`$this->assertTrue(is_dir($targetPath));`
`907`	`907`	`$this->assertFileEquals($sourcePath.'/nested/file1.txt', $targetPath.DIRECTORY_SEPARATOR.'link1/file1.txt');`
`908`	`908`	`$this->assertTrue(is_link($targetPath.DIRECTORY_SEPARATOR.'link1'));`
`909`		`- $this->assertEquals($sourcePath.'nested', readlink($targetPath.DIRECTORY_SEPARATOR.'link1'));`
	`909`	`+ $this->assertEquals('nested', readlink($targetPath.DIRECTORY_SEPARATOR.'link1'));`
`910`	`910`	`}`
`911`	`911`
`912`	`912`	`/**`