Skip to content
Snippets Groups Projects
Commit cceb3cf3 authored by Ruben Heinrich's avatar Ruben Heinrich
Browse files

[base] Improved path validation

parent c9b83b1a
No related branches found
No related tags found
1 merge request!2Basic import and export functionality
...@@ -8,13 +8,7 @@ scrape_benchmarks <- function( ...@@ -8,13 +8,7 @@ scrape_benchmarks <- function(
url = "https://gitlab.opengeosys.org/ogs/ogs/-/tree/master/Tests/Data/", url = "https://gitlab.opengeosys.org/ogs/ogs/-/tree/master/Tests/Data/",
path = "extdata/benchmarks/") { path = "extdata/benchmarks/") {
assertthat::assert_that(assertthat::is.string(path)) path <- validate_is_dir_path(path)
last_char_index <- length(path)
if(substring(path, last_char_index, last_char_index) != "/"){
path <- paste0(path, "/")
}
data_page <- xml2::read_html(url) data_page <- xml2::read_html(url)
...@@ -40,13 +34,8 @@ scrape_benchmarks <- function( ...@@ -40,13 +34,8 @@ scrape_benchmarks <- function(
download_benchmark <- function(prj_url, path) { download_benchmark <- function(prj_url, path) {
assertthat::assert_that(assertthat::is.string(prj_url)) assertthat::assert_that(assertthat::is.string(prj_url))
assertthat::assert_that(assertthat::is.string(path))
last_char_index <- length(path) path <- validate_is_dir_path(path)
if(substring(path, last_char_index, last_char_index) != "/"){
path <- paste0(path, "/")
}
prj_dest_file <- paste0(path, basename(prj_url)) prj_dest_file <- paste0(path, basename(prj_url))
file.create(prj_dest_file) file.create(prj_dest_file)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment