]> source.dussan.org Git - nextcloud-server.git/commitdiff
Mention missing files subdir 32874/head
authorBill McGonigle <bill-github.com@bfccomputing.com>
Tue, 14 Jun 2022 11:10:29 +0000 (07:10 -0400)
committerGitHub <noreply@github.com>
Tue, 14 Jun 2022 11:10:29 +0000 (07:10 -0400)
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

index 0a7a53dc0bf7e29baee1fde450221b47e9dfeeec..b40e963efc6df3d97cc30a01a688d2a607c8aa26 100644 (file)
@@ -144,7 +144,7 @@ class Scan extends Base {
                                $scanner->scan($path, $recursive, $homeOnly ? [$this, 'filterHomeMount'] : null);
                        }
                } catch (ForbiddenException $e) {
-                       $output->writeln("<error>Home storage for user $user not writable</error>");
+                       $output->writeln("<error>Home storage for user $user not writable or 'files' subdirectory missing</error>");
                        $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