diff --git a/NEWS.md b/NEWS.md index 5ad7a8f..569f39f 100644 --- a/NEWS.md +++ b/NEWS.md @@ -1,13 +1,17 @@ # arcgislayers (development version) -## Breaking changes - -- `dplyr` methods for `collect()`, `select()`, and `filter()` have been removed. - ## New features - Improve handling of `filter_geom` by `arc_select()` by warning if applying `sf::st_union()` to the filter does not generate a length 1 sfc, or if `filter_geom` is supplied when accessing a Table, or if `filter_geom` is empty (@elipousson, #166) +## Bug fixes + +- `arc_select()` includes argument name in error message when `...` contains non-string values. + +## Breaking changes + +- `dplyr` methods for `collect()`, `select()`, and `filter()` have been removed. + # arcgislayers 0.3.1 ## Bug fixes diff --git a/R/arc-select.R b/R/arc-select.R index 5c923ca..f279ab2 100644 --- a/R/arc-select.R +++ b/R/arc-select.R @@ -101,7 +101,7 @@ arc_select <- function( key <- dots_names[i] val <- dots[[i]] # check that the value is a scalar and non-empty - check_string(val, allow_empty = FALSE) + check_string(val, arg = key, allow_empty = FALSE) # insert into query query[[key]] <- val