Commons:User scripts/users namespace/reports/User:WilyD/monobook.js

This is the report page for User:WilyD/monobook.js. It is bot-maintained. All manual changes might be overwritten.

jshint

1. ISSUE: line 17 character 24: Use '===' to compare with 'false'. - Evidence: if(window.nfd_autosave == false){}else if(window.nfd_autosave){}else{ nfd_autosave = true; }
2. ISSUE: line 18 character 24: Use '===' to compare with 'false'. - Evidence: if(window.mnx_autosave == false){}else if(window.mnx_autosave){}else{ mnx_autosave = true; }
3. ISSUE: line 55 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function addLink(where, url1, name1, id, title, key, after, url2, name2)
4. ISSUE: line 86 character 15: 'na' is already defined. - Evidence: var na = document.createElement('a');
5. ISSUE: line 109 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function openWindow(url) {
6. ISSUE: line 114 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function getUploader() {
7. ISSUE: line 118 character 50: Missing semicolon. - Evidence: var el = document.getElementById('filehistory')
8. ISSUE: line 148 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function nfd_nomForDel() {
9. ISSUE: line 165 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function nfd_addDeleteTemplate() {
10. ISSUE: line 173 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function nfd_addIdwTemplate(target) {
11. ISSUE: line 182 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function nfd_updateDelReq(target, reason) {
12. ISSUE: line 190 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function nfd_updateDelReq2(target) {
13. ISSUE: line 198 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function nfd_onload() {
14. ISSUE: line 200 character 49: Script URL. - Evidence: addLink('p-tb', 'javascript:nfd_nomForDel()', nfd_text, 'nom-for-del', nfd_tooltip);
15. ISSUE: line 214 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function mnx_mark(imagepage_fakeaction, usertalk_fakeaction, message_lang) {
16. ISSUE: line 230 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function mnx_addTemplate(template, sorl) {
17. ISSUE: line 247 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function mnx_addUserWarningTemplate(imagetarget, template_type) {
18. ISSUE: line 253 character 14: 'txt' is already defined. - Evidence: var txt = '{{subst:image source' + (getParamValue('mnx_lang') ? '/' + getParamValue('mnx_lang') : '' ) + '|' + imagetarget + '}}';
19. ISSUE: line 256 character 48: 'txt' used out of scope. - Evidence: document.editform.wpTextbox1.value += '\n' + txt + '~~' + '~~\n';
20. ISSUE: line 263 character 1: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function mnx_onload() {
21. ISSUE: line 265 character 77: Script URL. - Evidence: addLink('p-tb', 'javascript:mnx_mark(\'mns_mns\', \'mnx_warn\', \'en\')', mns_text, 'mark-no-source', mns_tooltip, null, null, 'javascript:mnx_mark(\'mns_mns\', \'mnx_warn\', prompt(\'' + mnx_langquery + '\', wgUserLanguage))', mnx_lang);
22. ISSUE: line 265 character 190: Script URL. - Evidence: addLink('p-tb', 'javascript:mnx_mark(\'mns_mns\', \'mnx_warn\', \'en\')', mns_text, 'mark-no-source', mns_tooltip, null, null, 'javascript:mnx_mark(\'mns_mns\', \'mnx_warn\', prompt(\'' + mnx_langquery + '\', wgUserLanguage))', mnx_lang);
23. ISSUE: line 266 character 77: Script URL. - Evidence: addLink('p-tb', 'javascript:mnx_mark(\'mnp_mnp\', \'mnx_warn\', \'en\')', mnp_text, 'mark-no-permission', mnp_tooltip, null, null, 'javascript:mnx_mark(\'mnp_mnp\', \'mnx_warn\', prompt(\'' + mnx_langquery + '\', wgUserLanguage))', mnx_lang);
24. ISSUE: line 266 character 194: Script URL. - Evidence: addLink('p-tb', 'javascript:mnx_mark(\'mnp_mnp\', \'mnx_warn\', \'en\')', mnp_text, 'mark-no-permission', mnp_tooltip, null, null, 'javascript:mnx_mark(\'mnp_mnp\', \'mnx_warn\', prompt(\'' + mnx_langquery + '\', wgUserLanguage))', mnx_lang);
25. ISSUE: line 267 character 77: Script URL. - Evidence: addLink('p-tb', 'javascript:mnx_mark(\'mnl_mnl\', \'mnx_warn\', \'en\')', mnl_text, 'mark-no-license', mnl_tooltip, null, null, 'javascript:mnx_mark(\'mnl_mnl\', \'mnx_warn\', prompt(\'' + mnx_langquery + '\', wgUserLanguage))', mnx_lang);
26. ISSUE: line 267 character 191: Script URL. - Evidence: addLink('p-tb', 'javascript:mnx_mark(\'mnl_mnl\', \'mnx_warn\', \'en\')', mnl_text, 'mark-no-license', mnl_tooltip, null, null, 'javascript:mnx_mark(\'mnl_mnl\', \'mnx_warn\', prompt(\'' + mnx_langquery + '\', wgUserLanguage))', mnx_lang);