Fix variables, globals

worker.py

@@ -11,6 +11,7 @@ logger = Logger()
 
 
 def trigger_export(courseid, categoryid):
+    global MOODLE_PATH, OUTPUT_PATH
     output_name = f"course_{courseid}_{categoryid}_{datetime.now().timestamp()}"
     subprocess.run([MOOSH_PATH, "-p", MOODLE_PATH, "course-backup",
                    "-p", OUTPUT_PATH, "-f", output_name, "--template", courseid])
@@ -18,23 +19,24 @@ def trigger_export(courseid, categoryid):
 
 
 def main():
+    global SIGNAL_PATH, OUTPUT_PATH
     logger.info("Moodle Export Worker starting")
     SIGNAL_PATH = Path(SIGNAL_PATH)
 
-    if test.exists():
-        if not test.is_dir():
+    if SIGNAL_PATH.exists():
+        if not SIGNAL_PATH.is_dir():
             logger.fatal(
                 f"Cannot create signalling directory {SIGNAL_PATH} because a file with that name exists.")
             exit(1)
     else:
         try:
-            test.mkdir()
+            SIGNAL_PATH.mkdir()
         except Exception as e:
             logger.fatal(
                 f"Cannot create signalling directory {SIGNAL_PATH}: {e}")
             exit(1)
 
-    for f in test.glob("*-*.mew"):
+    for f in SIGNAL_PATH.glob("*-*.mew"):
         logger.debug(f"Found file {f.name} - start processing")
         try:
             pre = datetime.now()
@@ -48,6 +50,8 @@ def main():
             logger.error(
                 f"Export of course {f.name.split('.')[0].split('-')[0]} failed: {e}")
 
+    logger.info("Moodle Export Worker done")
+
 
 if __name__ == "__main__":
-    main()
+    main()