Skip to content

Commit 77870a5

Browse files
committed
minor symfony#40608 [Filesystem] Fix dumpFile() method call (sebpacz)
This PR was merged into the 4.4 branch. Discussion ---------- [Filesystem] Fix dumpFile() method call | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | ? | New feature? | no | Deprecations? | no | Tickets | --- | License | MIT | Doc PR | --- The dumpFile() method in the Filesystem component takes two arguments (filename and content). I found a test with the third argument set to null. Commits ------- 2b33d4a [Filesystem] Fix dumpFile() method call
2 parents 1a27291 + 2b33d4a commit 77870a5

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1731,7 +1731,7 @@ public function testDumpKeepsExistingPermissionsWhenOverwritingAnExistingFile()
17311731
file_put_contents($filename, 'FOO BAR');
17321732
chmod($filename, 0745);
17331733

1734-
$this->filesystem->dumpFile($filename, 'bar', null);
1734+
$this->filesystem->dumpFile($filename, 'bar');
17351735

17361736
$this->assertFilePermissions(745, $filename);
17371737
}

0 commit comments

Comments
 (0)