fix: Fix bug with referencing an out of scope variable

hotosm · Nov 8, 2023 · 40bd40c · 40bd40c
1 parent 7de0f6a
commit 40bd40c
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/osm_fieldwork/basemapper.py b/osm_fieldwork/basemapper.py
@@ -101,6 +101,7 @@ def dlthread(
             # Create the subdirectories as pySmartDL doesn't do it for us
             Path(dest).mkdir(parents=True, exist_ok=True)
 
+        dl = None
         try:
             if site["source"] == "topo":
                 filespec += "." + site["suffix"]
@@ -112,7 +113,10 @@ def dlthread(
                 log.debug("%s exists!" % (outfile))
         except Exception as e:
             log.error(e)
-            log.error("Couldn't download from %r: %s" % (filespec, dl.get_errors()))
+            if dl:
+                log.error(f"Couldn't download {filespec}: {dl.get_errors()}")
+            else:
+                log.error(f"Couldn't download {filespec}")
 
 
 class BaseMapper(object):