Commons:User scripts/users namespace/reports/User:1Veertje/ArworkRef-test.js

This is the report page for User:1Veertje/ArworkRef-test.js. It is bot-maintained. All manual changes might be overwritten.

jshint

1. ISSUE: line 218 character 43: Expected a conditional expression and instead saw an assignment. - Evidence: if (node = node.firstChild) do {
2. ISSUE: line 224 character 49: Expected a conditional expression and instead saw an assignment. - Evidence: } while (node = node.nextSibling);
3. ISSUE: line 229 character 14: Unnecessary semicolon. - Evidence: };
4. ISSUE: line 336 character 33: ['lang'] is better written in dot notation. - Evidence: var lang = htmlEl[0]['lang'] || htmlEl[0]['xml:lang'];
5. ISSUE: line 339 character 20: ['title'] is better written in dot notation. - Evidence: metaContent['title'] = document.title;
6. ISSUE: line 456 character 12: Missing semicolon. - Evidence: "}"
7. ISSUE: line 500 character 85: Missing semicolon. - Evidence: institution = institution + "|Institution:"+institution.split('The ')[1]
8. ISSUE: line 527 character 12: Missing semicolon. - Evidence: "}"
9. ISSUE: line 553 character 51: Use '===' to compare with ''. - Evidence: if (t.accessDate && t.dateFormatRetrieved == '')
10. ISSUE: line 596 character 47: Missing semicolon. - Evidence: authorParamNames.push(t.first + n)
11. ISSUE: line 604 character 20: 'n' is already defined. - Evidence: for (var n = 0; n <= 9; n++)
12. ISSUE: line 605 character 53: Use '===' to compare with '0'. - Evidence: names += (val.match(strt + t.first + (n == 0 ? '' : n) + end) || ['', ''])[1] + ' '
13. ISSUE: line 606 character 62: Use '===' to compare with '0'. - Evidence: + trimCollapse((val.match(strt + t.last + (n == 0 ? '' : n) + end) || ['', ''])[1]).replace(/ /g, '_') + ',';
14. ISSUE: line 618 character 28: 'n' is already defined. - Evidence: for (var n = 0; n < authorParamNames.length; n++)
15. ISSUE: line 760 character 87: ['copyFailed'] is better written in dot notation. - Evidence: if (copyFailed) this.parentNode.insertBefore(dom.textNode(msgs['copyFailed']), this.nextSibling);
16. ISSUE: line 1099 character 40: Use '===' to compare with '0'. - Evidence: if (siteLang.search(/^en-US/i) == 0) siteLang = 'en-US';
17. ISSUE: line 1290 character 35: Bad or unnecessary escaping. - Evidence: var regex = new RegExp('([0-9\.,]+)');
18. ISSUE: line 1302 character 33: Bad or unnecessary escaping. - Evidence: var regex = new RegExp('(ca\.|um|c\.)? ?(1[0-9]{3}) ?. ?(ca\.|um|c\.)? ?(1[0-9]{3})');
19. ISSUE: line 1302 character 40: Bad or unnecessary escaping. - Evidence: var regex = new RegExp('(ca\.|um|c\.)? ?(1[0-9]{3}) ?. ?(ca\.|um|c\.)? ?(1[0-9]{3})');
20. ISSUE: line 1302 character 65: Bad or unnecessary escaping. - Evidence: var regex = new RegExp('(ca\.|um|c\.)? ?(1[0-9]{3}) ?. ?(ca\.|um|c\.)? ?(1[0-9]{3})');
21. ISSUE: line 1302 character 72: Bad or unnecessary escaping. - Evidence: var regex = new RegExp('(ca\.|um|c\.)? ?(1[0-9]{3}) ?. ?(ca\.|um|c\.)? ?(1[0-9]{3})');
22. ISSUE: line 1322 character 29: Bad or unnecessary escaping. - Evidence: regex = new RegExp('(ca\.|um|c\.) ?(1[0-9]{3})');
23. ISSUE: line 1322 character 36: Bad or unnecessary escaping. - Evidence: regex = new RegExp('(ca\.|um|c\.) ?(1[0-9]{3})');
24. ISSUE: line 1338 character 74: Bad or unnecessary escaping. - Evidence: regex = new RegExp('(.*)([^0-9]|^)(1[0-9])(\b|th)( ([cC]entury|[Cc])\.?)?(.*)?');