Merge pull request #11 from pelias/fix-download-on-empty-config

fix(download): avoid error with empty config
pelias · Jan 22, 2019 · e6cced6 · e6cced6
2 parents f428ff8 + eee8fcc
commit e6cced6
Showing 1 changed file with 8 additions and 11 deletions.
diff --git a/utils/download_data.js b/utils/download_data.js
@@ -29,17 +29,14 @@ const files = config.get('imports.csv.download');
 
 if (!files) {
   logger.warn('No files to download, quitting');
-}
-
-if (!targetDir) {
+} else if (!targetDir) {
   logger.warn('Datapath for saving files not configured, quitting');
-}
+} else {
+  logger.info(`Attempting to download selected data files: ${files}`);
 
-logger.info(`Attempting to download selected data files: ${files}`);
+  fs.mkdirpSync(targetDir);
 
-
-fs.mkdirpSync(targetDir);
-
-async.eachLimit(files, 5, downloadStandard.bind(null, targetDir), function() {
-  logger.info('all done');
-});
+  async.eachLimit(files, 5, downloadStandard.bind(null, targetDir), function() {
+    logger.info('all done');
+  });
+}