merged branch deeky666/fix-filesystem-exception-message (PR symfony#8587)

fabpot · fabpot · commit 728d194d23fb · 2013-08-09T09:26:54.000+02:00
This PR was squashed before being merged into the master branch (closes symfony#8587). Discussion ---------- [Filesystem] fixed exception message when not a able to write to a directory | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | n/a | License | MIT | Doc PR | n/a This simply fixes an exception message in the Filesystem Component when a directory is not writable while dumping a file via `dumpFile()`. Commits ------- 8b32a4b [Filesystem] fixed exception message when not a able to write to a directory
diff --git a/src/Symfony/Component/Filesystem/Filesystem.php b/src/Symfony/Component/Filesystem/Filesystem.php
@@ -456,7 +456,7 @@ public function dumpFile($filename, $content, $mode = 0666)
         if (!is_dir($dir)) {
             $this->mkdir($dir);
         } elseif (!is_writable($dir)) {
-            throw new IOException(sprintf('Unable to write in the %s directory\n', $dir));
+            throw new IOException(sprintf('Unable to write to the "%s" directory.', $dir));
         }
 
         $tmpFile = tempnam($dir, basename($filename));

Original file line number	Diff line number	Diff line change
`@@ -456,7 +456,7 @@ public function dumpFile($filename, $content, $mode = 0666)`
`456`	`456`	`if (!is_dir($dir)) {`
`457`	`457`	`$this->mkdir($dir);`
`458`	`458`	`} elseif (!is_writable($dir)) {`
`459`		`- throw new IOException(sprintf('Unable to write in the %s directory\n', $dir));`
	`459`	`+ throw new IOException(sprintf('Unable to write to the "%s" directory.', $dir));`
`460`	`460`	`}`
`461`	`461`
`462`	`462`	`$tmpFile = tempnam($dir, basename($filename));`