Patch for stripping whitespace or "doi:" from DOI lookups
When I use the "Add item by identifier" (magic wand) function on a DOI it's annoying that it doesn't strip out any leading whitespace or "doi:".
That can be fixed in lookup.js by changing line 6+ from
if(identifier.substr(0, 3) == "10.") {
var item = {itemType:"journalArticle", DOI:identifier};
to
var doistart = identifier.indexOf("10.")
if(doistart >= 0) {
var doi = identifier.substr(doistart)
var item = {itemType:"journalArticle", DOI:doi};
This doesn't change the handling of ISBN/PMIDs ("10." is not valid in either) and it does the trick. If no one protests, can someone with access apply the patch?
That can be fixed in lookup.js by changing line 6+ from
if(identifier.substr(0, 3) == "10.") {
var item = {itemType:"journalArticle", DOI:identifier};
to
var doistart = identifier.indexOf("10.")
if(doistart >= 0) {
var doi = identifier.substr(doistart)
var item = {itemType:"journalArticle", DOI:doi};
This doesn't change the handling of ISBN/PMIDs ("10." is not valid in either) and it does the trick. If no one protests, can someone with access apply the patch?
-
dstillmanI've applied a variation of this that uses a utility function we have that pulls a DOI out of a string using a regex regardless of what's around it. Thanks.
This is an old discussion that has not been active in a long time. Instead of commenting here, you should start a new discussion. If you think the content of this discussion is still relevant, you can link to it from your new discussion.