symfony
diff --git a/‎src/Symfony/Component/Filesystem/Filesystem.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Filesystem/Filesystem.php
+2-2Lines changed: 2 additions & 2 deletions b/‎src/Symfony/Component/Filesystem/Filesystem.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Filesystem/Filesystem.php
+2-2Lines changed: 2 additions & 2 deletions
diff --git a/‎src/Symfony/Component/Filesystem/Tests/FilesystemTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Filesystem/Tests/FilesystemTest.php
+13Lines changed: 13 additions & 0 deletions b/‎src/Symfony/Component/Filesystem/Tests/FilesystemTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/Filesystem/Tests/FilesystemTest.php
+13Lines changed: 13 additions & 0 deletions
@@ -516,8 +516,8 @@ public function dumpFile($filename, $content, $mode = 0666)
             }
 
             $this->chmod($tmpFile, $mode);
-        } else {
-            @chmod($tmpFile, file_exists($filename) ? fileperms($filename) : 0666 & ~umask());
+        } elseif (file_exists($filename)) {
+            @chmod($tmpFile, fileperms($filename));
         }
 
         $this->rename($tmpFile, $filename, true);
 
@@ -1050,6 +1050,19 @@ public function testDumpFileOverwritesAnExistingFile()
         $this->assertSame('bar', file_get_contents($filename));
     }
 
+    public function testDumpKeepsExistingPermissionsWhenOverwritingAnExistingFile()
+    {
+        $this->markAsSkippedIfChmodIsMissing();
+
+        $filename = $this->workspace.DIRECTORY_SEPARATOR.'foo.txt';
+        file_put_contents($filename, 'FOO BAR');
+        chmod($filename, 0745);
+
+        $this->filesystem->dumpFile($filename, 'bar', null);
+
+        $this->assertFilePermissions(745, $filename);
+    }
+
     public function testCopyShouldKeepExecutionPermission()
     {
         $this->markAsSkippedIfChmodIsMissing();
Original file line number	Diff line number	Diff line change
`@@ -516,8 +516,8 @@ public function dumpFile($filename, $content, $mode = 0666)`
`516`	`516`	`}`
`517`	`517`
`518`	`518`	`$this->chmod($tmpFile, $mode);`
`519`		`- } else {`
`520`		`- @chmod($tmpFile, file_exists($filename) ? fileperms($filename) : 0666 & ~umask());`
	`519`	`+ } elseif (file_exists($filename)) {`
	`520`	`+ @chmod($tmpFile, fileperms($filename));`
`521`	`521`	`}`
`522`	`522`
`523`	`523`	`$this->rename($tmpFile, $filename, true);`