search

eblondel / zen4R

zen4R - R Interface to Zenodo REST API
https://github.com/eblondel/zen4R/wiki
Other
44 stars 14 forks source link

getDepositionByDOI() errors when Zenodo returns a 404 #76

Closed ThierryO closed 2 years ago

ThierryO commented 2 years ago

I'm getting Error in result$doi : $ operator is invalid for atomic vectors

zenodo <- ZenodoManager$new(token = key_get("zenodo"))
zenodo$getDepositionByDOI("10.5281/zenodo.5792949")

The line below fails because result <- list(message = "The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.", status = 404L)

https://github.com/eblondel/zen4R/blob/7320f3c89733c2752821e49bf3686913fefa4b41/R/ZenodoManager.R#L536

eblondel commented 2 years ago

thanks @ThierryO will look at it asap, first to see if I can reproduce it. Is it something that you experiment permanently? I'm asking because it looks like a temporary error.

ThierryO commented 2 years ago

This code used to work back in January. I'm trying to upload a new version of an existing deposit.

eblondel commented 2 years ago

Yes, actually, I had followed-up on a server-side change in Zenodo API (see https://github.com/eblondel/zen4R/issues/72), but now I realize they reverted it back... I'm going to applpy a fix for that

eblondel commented 2 years ago

@ThierryO this should be fixed, can you test it again?