fix #186 fix #187

pull out SI get fxns
pull out all remaining httr code
depends on dev crul version with intermediate headers

DESCRIPTION

@@ -26,8 +26,7 @@ Language: en-US
 VignetteBuilder: knitr
 Roxygen: list(wrap = FALSE, markdown = TRUE)
 Imports:
-    crul (>= 0.4.0),
-    httr (>= 1.3.1),
+    crul (>= 0.6.2.9334),
     magrittr,
     xml2 (>= 1.1.1),
     jsonlite,
@@ -42,12 +41,14 @@ Imports:
     pdftools,
     storr,
     tibble,
-    digest
+    digest,
+    httr
 Suggests:
     roxygen2 (>= 6.1.1),
     testthat,
     knitr
 RoxygenNote: 6.1.1
+Remotes: ropensci/crul@intermediate-headers
 X-schema.org-applicationCategory: Literature 
 X-schema.org-keywords: text-mining, literature, pdf, xml, publications, citations, full-text, TDM
 X-schema.org-isPartOf: https://ropensci.org

NAMESPACE

@@ -13,9 +13,6 @@ S3method(ft_get,default)
 S3method(ft_get,ft)
 S3method(ft_get,ft_links)
 S3method(ft_get,list)
-S3method(ft_get_si,character)
-S3method(ft_get_si,ft)
-S3method(ft_get_si,ft_data)
 S3method(ft_links,character)
 S3method(ft_links,ft)
 S3method(ft_links,ft_ind)
@@ -83,16 +80,6 @@ export(tabularize)
 importFrom(aRxiv,arxiv_search)
 importFrom(crul,HttpClient)
 importFrom(digest,digest)
-importFrom(httr,GET)
-importFrom(httr,HEAD)
-importFrom(httr,POST)
-importFrom(httr,content)
-importFrom(httr,content_type)
-importFrom(httr,stop_for_status)
-importFrom(httr,timeout)
-importFrom(httr,upload_file)
-importFrom(httr,write_disk)
-importFrom(jsonlite,fromJSON)
 importFrom(magrittr,"%>%")
 importFrom(microdemic,ma_abstract)
 importFrom(microdemic,ma_evaluate)

R/defunct.R

@@ -54,6 +54,14 @@ ft_browse_sections <- function(...) {
   .Defunct(msg = "function removed. see ?`fulltext-defunct`")
 }
 
+#' This function is defunct.
+#' @export
+#' @rdname ft_get_si-defunct
+#' @keywords internal
+ft_get_si <- function(...) {
+  .Defunct(msg = "function removed. see package suppdata")
+}
+
 
 #' Defunct functions in fulltext
 #'
@@ -69,6 +77,7 @@ ft_browse_sections <- function(...) {
 #'  - `cache_clear` was never working anyway, and is now removed
 #'  - [ft_browse_sections]: no sign that function used, and allows 
 #'  to remove a dependency
+#'  - [ft_get_si]: moved to package `suppdata`
 #' 
 #' @name fulltext-defunct
 NULL

R/ft_abstract.R

@@ -8,7 +8,7 @@
 #' @param scopusopts Scopus options, a named list.
 #' @param maopts Microsoft Academic options, a named list.
 #' @param crossrefopts Crossref options, a named list.
-#' @param ... curl options passed on to [httr::GET()] or [crul::HttpClient]
+#' @param ... curl options passed on to `httr::GET()` or [crul::HttpClient]
 #' @return An object of class `ft_abstract`
 #' @details See **Rate Limits** and **Authentication** in 
 #' [fulltext-package] for rate limiting and authentication information,