Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 1f7b753

Browse filesBrowse files
chalasrchalasr
committed
[Filesystem] Add appendToFile()
1 parent 8e497f2 commit 1f7b753
Copy full SHA for 1f7b753

File tree

Expand file treeCollapse file tree

3 files changed

+106
-1
lines changed
Filter options
Expand file treeCollapse file tree

3 files changed

+106
-1
lines changed

‎src/Symfony/Component/Filesystem/CHANGELOG.md

Copy file name to clipboardExpand all lines: src/Symfony/Component/Filesystem/CHANGELOG.md
+5Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,11 @@
11
CHANGELOG
22
=========
33

4+
3.3.0
5+
-----
6+
7+
* added `appendToFile()` to append contents to existing files
8+
49
3.2.0
510
-----
611

‎src/Symfony/Component/Filesystem/Filesystem.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Filesystem/Filesystem.php
+26-1Lines changed: 26 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -624,7 +624,7 @@ public function tempnam($dir, $prefix)
624624
* @param string $filename The file to be written to
625625
* @param string $content The data to write into the file
626626
*
627-
* @throws IOException If the file cannot be written to.
627+
* @throws IOException If the file cannot be written to
628628
*/
629629
public function dumpFile($filename, $content)
630630
{
@@ -648,6 +648,31 @@ public function dumpFile($filename, $content)
648648
$this->rename($tmpFile, $filename, true);
649649
}
650650

651+
/**
652+
* Appends content to an existing file.
653+
*
654+
* @param string $filename The file to which to append content
655+
* @param string $content The content to append
656+
*
657+
* @throws IOException If the file is not writable
658+
*/
659+
public function appendToFile($filename, $content)
660+
{
661+
$dir = dirname($filename);
662+
663+
if (!is_dir($dir)) {
664+
$this->mkdir($dir);
665+
}
666+
667+
if (!is_writable($dir)) {
668+
throw new IOException(sprintf('Unable to write to the "%s" directory.', $dir), 0, null, $dir);
669+
}
670+
671+
if (false === @file_put_contents($filename, $content, FILE_APPEND)) {
672+
throw new IOException(sprintf('Failed to write file "%s".', $filename), 0, null, $filename);
673+
}
674+
}
675+
651676
/**
652677
* @param mixed $files
653678
*

‎src/Symfony/Component/Filesystem/Tests/FilesystemTest.php

Copy file name to clipboardExpand all lines: src/Symfony/Component/Filesystem/Tests/FilesystemTest.php
+75Lines changed: 75 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1406,6 +1406,81 @@ public function testDumpFileWithZlibScheme()
14061406
$this->assertSame('bar', file_get_contents($filename));
14071407
}
14081408

1409+
public function testAppendToFile()
1410+
{
1411+
$filename = $this->workspace.DIRECTORY_SEPARATOR.'foo'.DIRECTORY_SEPARATOR.'bar.txt';
1412+
1413+
// skip mode check on Windows
1414+
if ('\\' !== DIRECTORY_SEPARATOR) {
1415+
$oldMask = umask(0002);
1416+
}
1417+
1418+
$this->filesystem->dumpFile($filename, 'foo');
1419+
1420+
$this->filesystem->appendToFile($filename, 'bar');
1421+
1422+
$this->assertFileExists($filename);
1423+
$this->assertSame('foobar', file_get_contents($filename));
1424+
1425+
// skip mode check on Windows
1426+
if ('\\' !== DIRECTORY_SEPARATOR) {
1427+
$this->assertFilePermissions(664, $filename, 'The written file should keep the same permissions as before.');
1428+
umask($oldMask);
1429+
}
1430+
}
1431+
1432+
public function testAppendToFileWithScheme()
1433+
{
1434+
if (defined('HHVM_VERSION')) {
1435+
$this->markTestSkipped('HHVM does not handle the file:// scheme correctly');
1436+
}
1437+
1438+
$scheme = 'file://';
1439+
$filename = $scheme.$this->workspace.DIRECTORY_SEPARATOR.'foo'.DIRECTORY_SEPARATOR.'baz.txt';
1440+
$this->filesystem->dumpFile($filename, 'foo');
1441+
1442+
$this->filesystem->appendToFile($filename, 'bar');
1443+
1444+
$this->assertFileExists($filename);
1445+
$this->assertSame('foobar', file_get_contents($filename));
1446+
}
1447+
1448+
public function testAppendToFileWithZlibScheme()
1449+
{
1450+
$scheme = 'compress.zlib://';
1451+
$filename = $this->workspace.DIRECTORY_SEPARATOR.'foo'.DIRECTORY_SEPARATOR.'baz.txt';
1452+
$this->filesystem->dumpFile($filename, 'foo');
1453+
1454+
// Zlib stat uses file:// wrapper so remove it
1455+
$this->assertSame('foo', file_get_contents(str_replace($scheme, '', $filename)));
1456+
1457+
$this->filesystem->appendToFile($filename, 'bar');
1458+
1459+
$this->assertFileExists($filename);
1460+
$this->assertSame('foobar', file_get_contents($filename));
1461+
}
1462+
1463+
public function testAppendToFileCreateTheFileIfNotExists()
1464+
{
1465+
$filename = $this->workspace.DIRECTORY_SEPARATOR.'foo'.DIRECTORY_SEPARATOR.'bar.txt';
1466+
1467+
// skip mode check on Windows
1468+
if ('\\' !== DIRECTORY_SEPARATOR) {
1469+
$oldMask = umask(0002);
1470+
}
1471+
1472+
$this->filesystem->appendToFile($filename, 'bar');
1473+
1474+
// skip mode check on Windows
1475+
if ('\\' !== DIRECTORY_SEPARATOR) {
1476+
$this->assertFilePermissions(664, $filename);
1477+
umask($oldMask);
1478+
}
1479+
1480+
$this->assertFileExists($filename);
1481+
$this->assertSame('bar', file_get_contents($filename));
1482+
}
1483+
14091484
public function testCopyShouldKeepExecutionPermission()
14101485
{
14111486
$this->markAsSkippedIfChmodIsMissing();

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.