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

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

jshint

1. ISSUE: line 39 character 6: Missing semicolon. - Evidence: }
2. ISSUE: line 55 character 107: Use '!==' to compare with '0'. - Evidence: if(document.title.indexOf("Wikipedia:Votes for deletion") != -1 && document.title.indexOf("Editing ") != 0) vfdlinks();
3. ISSUE: line 239 character 43: Script URL. - Evidence: addlilink(tabs, 'javascript:hidevfd()', 'hide closed', 'ca-hide');
4. ISSUE: line 241 character 43: Script URL. - Evidence: addlilink(tabs, 'javascript:showvfd()', 'show closed', 'ca-show');
5. ISSUE: line 270 character 12: Use '===' to compare with 'null'. - Evidence: if(r == null) return false;
6. ISSUE: line 284 character 71: Use '===' to compare with '0'. - Evidence: if(document.title.indexOf("Editing Wikipedia:Votes for deletion") == 0)
7. ISSUE: line 288 character 58: Script URL. - Evidence: addlilink(vfda, 'javascript:closevfd("keep", "")', 'keep', '');
8. ISSUE: line 289 character 120: Script URL. - Evidence: addlilink(vfda, 'javascript:closevfd("merge and redirect", " to [[" + prompt("Merge and redirect to?") + "]]")', 'merge', '');
9. ISSUE: line 290 character 100: Script URL. - Evidence: addlilink(vfda, 'javascript:closevfd("redirect", " to [[" + prompt("Redirect to?") + "]]")', 'redir', '');
10. ISSUE: line 291 character 93: Script URL. - Evidence: addlilink(vfda, 'javascript:closevfd("transwiki", " to " + prompt("Transwiki to?"))', 'trans', '');
11. ISSUE: line 292 character 77: Script URL. - Evidence: addlilink(vfda, 'javascript:closevfd("transwiki", " to Wiktionary")', 'wikt', '');
12. ISSUE: line 293 character 67: Script URL. - Evidence: addlilink(vfda, 'javascript:closevfd("speedy delete", "")', 'speedy', '');
13. ISSUE: line 294 character 60: Script URL. - Evidence: addlilink(vfda, 'javascript:closevfd("delete", "")', 'del', '');
14. ISSUE: line 295 character 69: Script URL. - Evidence: addlilink(vfda, 'javascript:closevfd(prompt("Result?"), "")', 'other', '');
15. ISSUE: line 296 character 47: Script URL. - Evidence: addlilink(tabs, 'javascript:replace()', 'replace', '');
16. ISSUE: line 298 character 63: Use '===' to compare with '0'. - Evidence: else if(document.title.indexOf("Confirm delete - Delete") == 0)
17. ISSUE: line 300 character 49: Script URL. - Evidence: addlilink(tabs, 'javascript:vfddelete()', 'vfd', '');
18. ISSUE: line 302 character 58: Use '===' to compare with '0'. - Evidence: else if(document.title.indexOf("Editing User talk:") == 0)
19. ISSUE: line 304 character 47: Script URL. - Evidence: addlilink(tabs, 'javascript:replace()', 'replace', '');
20. ISSUE: line 307 character 55: Script URL. - Evidence: addlilink(talkm, 'javascript:welcomeMessage()', 'welcome', '');
21. ISSUE: line 308 character 52: Script URL. - Evidence: addlilink(talkm, 'javascript:anonWelcome()', 'anon', '');
22. ISSUE: line 309 character 52: Script URL. - Evidence: addlilink(talkm, 'javascript:startThread()', 'thread', '');
23. ISSUE: line 310 character 52: Script URL. - Evidence: addlilink(talkm, 'javascript:editSummary()', 'summary', '');
24. ISSUE: line 312 character 48: Use '===' to compare with '0'. - Evidence: else if(document.title.indexOf("Editing ") == 0)
25. ISSUE: line 314 character 47: Script URL. - Evidence: addlilink(tabs, 'javascript:replace()', 'replace', '');
26. ISSUE: line 315 character 49: Script URL. - Evidence: addlilink(tabs, 'javascript:vfdresult()', 'vfd result', '');
27. ISSUE: line 340 character 15: Implied eval. Consider passing a function instead of a string. - Evidence: timerID = setTimeout('showtime()', 100);
28. ISSUE: line 377 character 14: Use '===' to compare with ''. - Evidence: if (target == '')
29. ISSUE: line 430 character 39: Use '===' to compare with '0'. - Evidence: if (document.title.indexOf('User:') == 0
30. ISSUE: line 431 character 47: Use '===' to compare with '0'. - Evidence: || document.title.indexOf('User talk:') == 0)
31. ISSUE: line 433 character 49: Use '===' to compare with '0'. - Evidence: else if (document.title.indexOf('Block user') == 0) // could stand to be more robust
32. ISSUE: line 444 character 5: 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 SoFixItInit() // pre-load, (don't want to slow down loading of article's content, though)
33. ISSUE: line 449 character 5: 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 SoFixItLoad() // post-load
34. ISSUE: line 496 character 101: Script URL. - Evidence: UserMenu.append( 'pt-utc', UTCTime(), 'javascript:void UserMenu.setText("pt-utc",UTCTime())' );
35. ISSUE: line 513 character 5: 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 PortletMenu( id ) // constructor
36. ISSUE: line 538 character 10: Missing semicolon. - Evidence: }
37. ISSUE: line 542 character 10: Missing semicolon. - Evidence: }
38. ISSUE: line 546 character 10: Missing semicolon. - Evidence: }
39. ISSUE: line 551 character 114: Missing semicolon. - Evidence: this.getText = function( id ) { return this[ id ].getElementsByTagName( 'a' )[ 0 ].firstChild.data }
40. ISSUE: line 552 character 120: Missing semicolon. - Evidence: this.setText = function( id, txt ) { this[ id ].getElementsByTagName( 'a' )[ 0 ].firstChild.data = txt }
41. ISSUE: line 554 character 109: Missing semicolon. - Evidence: this.getHref = function( id ) { return this[ id ].getElementsByTagName( 'a' )[ 0 ].href }
42. ISSUE: line 555 character 109: Missing semicolon. - Evidence: this.setHref = function( id, url ) { this[ id ].getElementsByTagName( 'a' )[ 0 ].href = url }
43. ISSUE: line 560 character 5: 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 RemoveNode( id )
44. ISSUE: line 561 character 50: Missing semicolon. - Evidence: { var node = document.getElementById( id )
45. ISSUE: line 565 character 5: 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 UTCTime()
46. ISSUE: line 1 character 11: Missing semicolon. - Evidence: showtime()