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

Journal-level ISSN pseudo-DOI #16

Open
nemobis opened this issue Aug 15, 2018 · 3 comments
Open

Journal-level ISSN pseudo-DOI #16

nemobis opened this issue Aug 15, 2018 · 3 comments

Comments

@nemobis
Copy link

nemobis commented Aug 15, 2018

Unless I'm mistaken, the recent dump includes a number of identifiers in the form 10.1002/%28ISSN%291099-0690 (escaped parentheses around "ISSN" in the original) which presumably come from templates such as {{Official website|http://onlinelibrary.wiley.com/journal/10.1002/%28ISSN%291099-0690}}.

These identifiers are used as prefixes of the actual DOIs for articles in those papers, but I'm not sure they're real DOIs, and at any rate not in the URL-escaped form. (http://doi.org/10.1002/(ISSN)1099-0690 does redirect to https://onlinelibrary.wiley.com/journal/10990690.)

doi-issn.txt

@kodchi
Copy link
Contributor

kodchi commented Aug 22, 2018

I wonder if it would be better to fix the issue at the template level.

@nemobis
Copy link
Author

nemobis commented Aug 22, 2018 via email

@kodchi
Copy link
Contributor

kodchi commented Aug 22, 2018

Yeah, why not replace %28 with ( at the core and not deal with it here? Ditto %29.

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

No branches or pull requests

2 participants