From 8a3c19f894ddbbe59ccd730c74e56d48910618c4 Mon Sep 17 00:00:00 2001 From: Bill McGonigle Date: Tue, 14 Jun 2022 07:10:29 -0400 Subject: [PATCH] Mention missing files subdir Many users are getting confused by the inaccurate error message "Home storage for user $user not writable" because the storage *is* writable. The actual issue is a missing files/ subdirectory. cf. https://help.nextcloud.com/t/home-storage-for-user-not-writable/10831/7 By mentioning the possible cause in the error message, users are going to be able to rapidly solve their problem rather than bang their heads against the screen, Google, and eventually forums to find out that the error message is wrong in their case. Yes, it would be better to detect and precisely describe the fault, or fix the problem automatically, but until then, be kind to the users for the next however many years. --- apps/files/lib/Command/Scan.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/files/lib/Command/Scan.php b/apps/files/lib/Command/Scan.php index ff96fbf2dabf4..70b8729861201 100644 --- a/apps/files/lib/Command/Scan.php +++ b/apps/files/lib/Command/Scan.php @@ -143,7 +143,7 @@ protected function scanFiles($user, $path, OutputInterface $output, $backgroundS $scanner->scan($path, $recursive, $homeOnly ? [$this, 'filterHomeMount'] : null); } } catch (ForbiddenException $e) { - $output->writeln("Home storage for user $user not writable"); + $output->writeln("Home storage for user $user not writable or 'files' subdirectory missing"); $output->writeln('Make sure you\'re running the scan command only as the user the web server runs as'); } catch (InterruptedException $e) { # exit the function if ctrl-c has been pressed