[Filesystem] Check that the directory is writable after created it in dumpFile()

chalasr · chalasr · commit dbc41485357b · 2017-01-08T13:55:49.000+01:00
diff --git a/src/Symfony/Component/Filesystem/Filesystem.php b/src/Symfony/Component/Filesystem/Filesystem.php
@@ -498,7 +498,9 @@ public function dumpFile($filename, $content, $mode = 0666)
 
         if (!is_dir($dir)) {
             $this->mkdir($dir);
-        } elseif (!is_writable($dir)) {
+        }
+
+        if (!is_writable($dir)) {
             throw new IOException(sprintf('Unable to write to the "%s" directory.', $dir), 0, null, $dir);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -498,7 +498,9 @@ public function dumpFile($filename, $content, $mode = 0666)`
`498`	`498`
`499`	`499`	`if (!is_dir($dir)) {`
`500`	`500`	`$this->mkdir($dir);`
`501`		`- } elseif (!is_writable($dir)) {`
	`501`	`+ }`
	`502`	`+`
	`503`	`+ if (!is_writable($dir)) {`
`502`	`504`	`throw new IOException(sprintf('Unable to write to the "%s" directory.', $dir), 0, null, $dir);`
`503`	`505`	`}`
`504`	`506`