update minimum rio version, remove unneeded logic

DESCRIPTION

@@ -1,7 +1,7 @@
 Package: boxr
 Type: Package
 Title: Interface for the 'Box.com API'
-Version: 0.3.6.9005
+Version: 0.3.6.9006
 Authors@R: c(
     person("Brendan", "Rocks", email = "[email protected]",
            role = c("aut")),
@@ -44,7 +44,7 @@ Imports:
     magrittr,
     mime,
     purrr,
-    rio,
+    rio (>= 0.5.18),
     rlang,
     stats,
     stringr,

NEWS.md

@@ -2,6 +2,8 @@
 
 ## Internal
 
+* update minimum version of rio package to reflect newer treatment of JSON files. (#261)
+
 * update documentation to reflect updates to rio package. (#242, @chainsawriot)
 
 * update maintainer's email address. (#248)

R/boxr_read.R

@@ -97,15 +97,6 @@ box_read <- function(file_id, type = NULL, version_id = NULL,
     cont <- read_fun(new_name, ...)
   }
 
-  # this code comment is old (i think) and maybe worth revisiting was rio goes to CRAN (NCD 2019-11-01)
-  # \/
-  # rio is imposing the data.frame class on .json files, which isn't lolz.
-  # So, if it's classed as a data.frame but doesn't have the 'row.names'
-  # attribute, unclass it
-  if ("data.frame" %in% class(cont) & is.null(attr(cont, "row.names"))) {
-    cont <- unclass(cont)
-  }
-
   # Delete the tempfile
   unlink(temp_file, force = TRUE)