Added system checks to warn people of misconfigurations

src/documents/management/commands/document_consumer.py

@@ -19,7 +19,8 @@ class Command(BaseCommand):
     LOOP_TIME = 10  # Seconds
     MAIL_DELTA = datetime.timedelta(minutes=10)
 
-    MEDIA_DOCS = os.path.join(settings.MEDIA_ROOT, "documents")
+    ORIGINAL_DOCS = os.path.join(settings.MEDIA_ROOT, "documents", "originals")
+    THUMB_DOCS = os.path.join(settings.MEDIA_ROOT, "documents", "thumbnails")
 
     def __init__(self, *args, **kwargs):
 
@@ -40,10 +41,11 @@ class Command(BaseCommand):
         except (ConsumerError, MailFetcherError) as e:
             raise CommandError(e)
 
-        try:
+        for path in (self.ORIGINAL_DOCS, self.THUMB_DOCS):
-            os.makedirs(self.MEDIA_DOCS)
+            try:
-        except FileExistsError:
+                os.makedirs(path)
-            pass
+            except FileExistsError:
+                pass
 
         logging.getLogger(__name__).info(
             "Starting document consumer at {}".format(settings.CONSUMPTION_DIR)

src/paperless/__init__.py

@@ -0,0 +1 @@
+from .checks import paths_check

src/paperless/checks.py

@@ -0,0 +1,47 @@
+import os
+
+from django.core.checks import Error, register, Warning
+
+
+@register()
+def paths_check(app_configs, **kwargs):
+
+    check_messages = []
+
+    exists_message = "{} is set but doesn't exist."
+    exists_hint = "Create a directory at {}"
+    writeable_message = "{} is not writeable"
+    writeable_hint = (
+        "Set the permissions of {} to be writeable by the user running the "
+        "Paperless services"
+    )
+
+    directory = os.getenv("PAPERLESS_DBDIR")
+    if directory:
+        if not os.path.exists(directory):
+            check_messages.append(Error(
+                exists_message.format("PAPERLESS_DBDIR"),
+                exists_hint.format(directory)
+            ))
+        if not check_messages:
+            if not os.access(directory, os.W_OK | os.X_OK):
+                check_messages.append(Error(
+                    writeable_message.format("PAPERLESS_DBDIR"),
+                    writeable_hint.format(directory)
+                ))
+
+    directory = os.getenv("PAPERLESS_MEDIADIR")
+    if directory:
+        if not os.path.exists(directory):
+            check_messages.append(Error(
+                exists_message.format("PAPERLESS_MEDIADIR"),
+                exists_hint.format(directory)
+            ))
+        if not check_messages:
+            if not os.access(directory, os.W_OK | os.X_OK):
+                check_messages.append(Error(
+                    writeable_message.format("PAPERLESS_MEDIADIR"),
+                    writeable_hint.format(directory)
+                ))
+
+    return check_messages