Merge pull request #63 from bcgov/fix-historic-removal

put historic db in subfolder, only remove db file. Fixes #62
bcgov · Apr 14, 2021 · de8939c · de8939c
2 parents 9fce384 + c652746
commit de8939c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/R/ems_sqlite.R b/R/ems_sqlite.R
@@ -65,7 +65,7 @@ download_historic_data <- function(force = FALSE, ask = TRUE, dont_update = FALS
   on.exit(unlink(csv_file), add = TRUE)
 
   if (db_exists) {
-    unlink(dirname(db_path), recursive = TRUE)
+    file.remove(db_path)
     write_db_path()
   }
 
@@ -277,7 +277,9 @@ stringify_vec <- function(vec) {
 
 write_db_path <- function(path = getOption("rems.historic.path",
                             default = rems_data_dir())) {
-  file.path(path, "ems_historic.sqlite")
+  db_path <- file.path(path, "historic_db", "ems_historic.sqlite")
+  dir.create(dirname(db_path), recursive = TRUE, showWarnings = FALSE)
+  db_path
 }
 
 #' Add an index to a column in a sqlite database