Skip to content

Commit 5b2cf3c

Browse files
committed
touch the file first, because a non existing file is not writable
Signed-off-by: Bernd Stellwag <burned@zerties.org>
1 parent e37e248 commit 5b2cf3c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

lib/private/Config.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,8 +240,8 @@ private function writeData() {
240240
// Check if config directory/file is writable and/or create the config file
241241
if (
242242
(!is_file($this->configFilePath) && !is_writable(dirname($this->configFilePath)))
243-
|| !is_writable($this->configFilePath)
244243
|| !touch($this->configFilePath)
244+
|| !is_writable($this->configFilePath)
245245
) {
246246
// TODO fix this via DI once it is very clear that this doesn't cause side effects due to initialization order
247247
// currently this breaks app routes but also could have other side effects especially during setup and exception handling

0 commit comments

Comments
 (0)