Skip to content

Commit 1b10137

Browse files
committed
Don't increment data link counter if Extra field is not a DOI
1 parent 9914426 commit 1b10137

File tree

1 file changed

+14
-7
lines changed

1 file changed

+14
-7
lines changed

zotero.js

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -152,18 +152,25 @@ function parseZoteroResults(resultText) {
152152
function parseDataLinks(extra) {
153153
if (extra) {
154154
var dois = extra.split(/\r?\n/);
155-
var links = []
155+
var urls = [];
156156
for (var i = 0; i < dois.length; i++) {
157157
var doi = dois[i];
158-
var j = i + 1
159158
if (doi.startsWith("https://doi.org/")) {
160-
if (dois.length == 1) {
161-
links.push(' <a href="' + doi + '" target="_blank" rel="noopener" aria-label="open data in new tab">Data link.</a>');
162-
} else {
163-
links.push(' <a href="' + doi + '" target="_blank" rel="noopener" aria-label="open data in new tab">Data link ' + j + '.</a>');
164-
}
159+
urls.push(doi);
165160
}
166161
}
162+
var links = [];
163+
if (urls.length == 0) {
164+
return "";
165+
} else if (urls.length == 1) {
166+
links.push(' <a href="' + urls[0] + '" target="_blank" rel="noopener" aria-label="open data in new tab">Data link.</a>');
167+
} else {
168+
for (var i = 0; i < urls.length; i++) {
169+
var url = urls[i];
170+
var j = i + 1;
171+
links.push(' <a href="' + url + '" target="_blank" rel="noopener" aria-label="open data in new tab">Data link ' + j + '.</a>');
172+
}
173+
}
167174
return links.join(" ");
168175
} else {
169176
return "";

0 commit comments

Comments
 (0)