R-ArcGIS / arcgislayers

ArcGIS Location Services
http://r.esri.com/arcgislayers/
Apache License 2.0
37 stars 8 forks source link

standalone checks & assert n_max is a whole positive number #181

Closed JosiahParry closed 16 hours ago

JosiahParry commented 2 months ago

Describe the bug n_max in arc_select() and arc_read() can be set to a negative number. This causes an error to be propagated with a confusing message. Instead, there should be an assertion like check_whole_number() to do this. arcgisutils has imported the standalone checks with proper rlang namespacing. These should be incorporated into arcgislayers to perform these checks