Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OUP Plugin fails on some OUP DOIs #73

Closed
cameronneylon opened this issue Apr 13, 2014 · 2 comments
Closed

OUP Plugin fails on some OUP DOIs #73

cameronneylon opened this issue Apr 13, 2014 · 2 comments
Assignees
Labels

Comments

@cameronneylon
Copy link
Contributor

This one for instance appears not to work:
http://dx.doi.org/10.1093/brain/awt099 although it seems to have the correct text.

A CC BY example: http://dx.doi.org/10.1093/bioinformatics/bts547

There are also some OUP journals that use a somewhat different text:
http://dx.doi.org/10.1093/cvr/cvt012 is an example

Some other recently failed DOIs:
10.1093/brain/awt148
10.1093/brain/aws268
10.1093/alcalc/agt006
10.1093/aje/kwt084
10.1093/cercor/bhs420

@cameronneylon
Copy link
Contributor Author

This is a result of a missing trailing slash in the URL for the NC license statement. For instance for these two articles in the same journal one successfully returns an NC license but the other doesn't match:

http://oag.cottagelabs.com/lookup/10.1093/schbul/sbs151,10.1093/schbul/sbs053

I'll add the additional license term to the OUP plugin and add a new test for these cases.

@cameronneylon
Copy link
Contributor Author

Closing as the new pull request should cover those cases that are listed in this issue at least.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant