search

muschellij2 / rscopus

Scopus Database API Interface to R
74 stars 16 forks source link

argument inconsistency in citation_retrieval compared to article_retrieval and abstract_retrieval #46

Closed emilBeBri closed 2 years ago

emilBeBri commented 2 years ago

I struggled quite a bit with citation_retrieval(), until I realized that the logic of the arguments is completely different:

in these two, you specify which type of identifier as a serate argument:

article_retrieval('10, identifier='doi', .1287/mnsc.2021.3997')
abstract_retrieval('10.1287/mnsc.2021.3997', identifier='doi')

but in citation_retreival, you have to use a named argument for the doi, for example:

citation_retrieval(doi='10.1287/mnsc.2021.3997')

this means that if you try to use the same logic in the other two:

citation_retrieval('10.1287/mnsc.2021.3997', identifier='doi')

it won't fail - but it will try to use the doi as an eid, and get an empty result back. This is very confusing.

muschellij2 commented 2 years ago

Pull requests are welcome to rectify these.

muschellij2 commented 2 years ago

ALso if you look at the API https://dev.elsevier.com/documentation/AbstractCitationAPI.wadl, it shows you can specify DOIs, PubMed IDs, and scopus IDs all in one call.