😓 return NULL when no email address is stored in .Renviron

ropensci · Oct 19, 2017 · d5e34dd · d5e34dd
1 parent aaab00f
commit d5e34dd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -7,10 +7,6 @@ addons:
     packages:
       - poppler-utils
 
-before_install:
-  - mkdir -p ~/Rlib
-  - echo 'crossref_email = name@example.com' > .Renviron
-
 r_github_packages:
   - jimhester/covr
 

diff --git a/R/zzz.R b/R/zzz.R
@@ -26,6 +26,7 @@ cr_GET <- function(endpoint, args, todf = TRUE, on_error = warning, parse = TRUE
   url <- sprintf("https://api.crossref.org/%s", endpoint)
   cli <- crul::HttpClient$new(
     url = url,
+    opts = list(verbose = TRUE),
     headers = list(
       `User-Agent` = rcrossref_ua(),
       `X-USER-AGENT` = rcrossref_ua()
@@ -165,8 +166,9 @@ get_email <- function() {
   email <- Sys.getenv("crossref_email")
   if (identical(email, "")) {
     NULL
-  }
+  } else {
   paste0("(mailto:", val_email(email), ")")
+  }
 }
 
 #' Email checker