[Filesystem] Add third argument $flags to Filesystem::appendToFile()

fwolfsjaeger · fwolfsjaeger · commit bc331f203f21 · 2021-10-28T11:23:36.000+02:00
diff --git a/src/Symfony/Component/Filesystem/CHANGELOG.md b/src/Symfony/Component/Filesystem/CHANGELOG.md
@@ -5,6 +5,7 @@ CHANGELOG
 ---
 
  * Add `Path` class
+ * `Filesystem::appendToFile()` now accepts a third argument `$flags`, which will be added to \FILE_APPEND
 
 5.0.0
 -----
diff --git a/src/Symfony/Component/Filesystem/Filesystem.php b/src/Symfony/Component/Filesystem/Filesystem.php
@@ -691,11 +691,13 @@ public function dumpFile(string $filename, $content)
     /**
      * Appends content to an existing file.
      *
+     * @param string $filename
      * @param string|resource $content The content to append
+     * @param int $flags
      *
      * @throws IOException If the file is not writable
      */
-    public function appendToFile(string $filename, $content)
+    public function appendToFile(string $filename, $content, int $flags = 0)
     {
         if (\is_array($content)) {
             throw new \TypeError(sprintf('Argument 2 passed to "%s()" must be string or resource, array given.', __METHOD__));
@@ -707,7 +709,7 @@ public function appendToFile(string $filename, $content)
             $this->mkdir($dir);
         }
 
-        if (false === self::box('file_put_contents', $filename, $content, \FILE_APPEND)) {
+        if (false === self::box('file_put_contents', $filename, $content, $flags | \FILE_APPEND)) {
             throw new IOException(sprintf('Failed to write file "%s": ', $filename).self::$lastError, 0, null, $filename);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -691,11 +691,13 @@ public function dumpFile(string $filename, $content)`
`691`	`691`	`/**`
`692`	`692`	`* Appends content to an existing file.`
`693`	`693`	`*`
	`694`	`+ * @param string $filename`
`694`	`695`	`* @param string\|resource $content The content to append`
	`696`	`+ * @param int $flags`
`695`	`697`	`*`
`696`	`698`	`* @throws IOException If the file is not writable`
`697`	`699`	`*/`
`698`		`- public function appendToFile(string $filename, $content)`
	`700`	`+ public function appendToFile(string $filename, $content, int $flags = 0)`
`699`	`701`	`{`
`700`	`702`	`if (\is_array($content)) {`
`701`	`703`	`throw new \TypeError(sprintf('Argument 2 passed to "%s()" must be string or resource, array given.', __METHOD__));`
`@@ -707,7 +709,7 @@ public function appendToFile(string $filename, $content)`
`707`	`709`	`$this->mkdir($dir);`
`708`	`710`	`}`
`709`	`711`
`710`		`- if (false === self::box('file_put_contents', $filename, $content, \FILE_APPEND)) {`
	`712`	`+ if (false === self::box('file_put_contents', $filename, $content, $flags \| \FILE_APPEND)) {`
`711`	`713`	`throw new IOException(sprintf('Failed to write file "%s": ', $filename).self::$lastError, 0, null, $filename);`
`712`	`714`	`}`
`713`	`715`	`}`