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

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

jshint

1. ISSUE: line 2 character 1: document.write can be a form of eval. - Evidence: document.write('<script type="text/javascript" src="'
2. ISSUE: line 17 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 query_string_get(paramName) {
3. ISSUE: line 21 character 22: Expected a conditional expression and instead saw an assignment. - Evidence: if (m=cmdRe.exec(h)) {
4. ISSUE: line 27 character 2: Unnecessary semicolon. - Evidence: };
5. ISSUE: line 30 character 68: Script URL. - Evidence: mw.util.addPortletLink('p-cactions', "javascript:prepare_for_autodel()", "s", 'ca-autodellinks', 'Alter all the links on this page to autodel the page when clicked');
6. ISSUE: line 33 character 32: Use '!==' to compare with 'null'. - Evidence: if(query_string_get('autodel') != null && query_string_get('action') == 'delete') {
7. ISSUE: line 53 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 prepare_for_autodel() {
8. ISSUE: line 55 character 12: Use '===' to compare with 'null'. - Evidence: if(tag == null) { alert('Action annulée.'); return; }
9. ISSUE: line 56 character 12: Use '===' to compare with ''. - Evidence: if(tag == "") tag = autodel.default_summary;
10. ISSUE: line 71 character 28: Use '===' to compare with ''. - Evidence: if(linkpage[1] == '') {
11. ISSUE: line 83 character 69: Don't make functions within a loop. - Evidence: nli.onmouseup = function() { this.style.color = 'red'; };
12. ISSUE: line 89 character 66: Don't make functions within a loop. - Evidence: link.onmouseup = function() { this.style.color = 'red'; };
13. ISSUE: line 93 character 40: 'i' is already defined. - Evidence: if(autodel.delete_talks) for(var i = 0; i < newlinks.length; i++) {
14. ISSUE: line 103 character 53: Script URL. - Evidence: btn.href = 'javascript:unprepare_for_autodel();';
15. ISSUE: line 108 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 unprepare_for_autodel() {
16. ISSUE: line 112 character 51: Don't make functions within a loop. - Evidence: link.onclick = function() { return true; };
17. ISSUE: line 113 character 53: Don't make functions within a loop. - Evidence: link.onmouseup = function() { return true; };
18. ISSUE: line 123 character 51: Script URL. - Evidence: btn.href = 'javascript:prepare_for_autodel();';
19. ISSUE: line 203 character 33: Bad or unnecessary escaping. - Evidence: str = str.replace(/ \./g, "\.");
20. ISSUE: line 206 character 39: Bad or unnecessary escaping. - Evidence: str = str.replace(/\.\.\.\.+/g, "\.\.\.");
21. ISSUE: line 206 character 41: Bad or unnecessary escaping. - Evidence: str = str.replace(/\.\.\.\.+/g, "\.\.\.");
22. ISSUE: line 206 character 43: Bad or unnecessary escaping. - Evidence: str = str.replace(/\.\.\.\.+/g, "\.\.\.");
23. ISSUE: line 207 character 41: Bad or unnecessary escaping. - Evidence: str = str.replace(/etc\.\.+/g, "etc\.");
24. ISSUE: line 241 character 41: Bad or unnecessary escaping. - Evidence: str = str.replace(/\[\[(.*?) —/g, "\[\[$1 -");
25. ISSUE: line 241 character 43: Bad or unnecessary escaping. - Evidence: str = str.replace(/\[\[(.*?) —/g, "\[\[$1 -");
26. ISSUE: line 242 character 41: Bad or unnecessary escaping. - Evidence: str = str.replace(/\|( *)—( *)/g, "\|");
27. ISSUE: line 259 character 24: Use '===' to compare with '0'. - Evidence: if (b.indexOf(a) == 0 || b.indexOf(TurnFirstToLower(a)) == 0)
28. ISSUE: line 259 character 63: Use '===' to compare with '0'. - Evidence: if (b.indexOf(a) == 0 || b.indexOf(TurnFirstToLower(a)) == 0)
29. ISSUE: line 280 character 13: Use '!==' to compare with ''. - Evidence: if (input != "")
30. ISSUE: line 282 character 15: 'input' is already defined. - Evidence: var input = trim(input);
31. ISSUE: line 292 character 55: Script URL. - Evidence: mw.util.addPortletLink('p-personal', 'javascript:format()', 'typo', 'ca-format', 'Corrections de la mise en page et de la typographie', '', document.getElementById('ca-edit'));
32. ISSUE: line 301 character 1: document.write can be a form of eval. - Evidence: document.write('<script type="text/javascript" src="'
33. ISSUE: line 306 character 1: document.write can be a form of eval. - Evidence: document.write('<script type="text/javascript" src="'
34. ISSUE: line 346 character 1: document.write can be a form of eval. - Evidence: document.write('<script type="text/javascript" src="'
35. ISSUE: line 579 character 13: Use '===' to compare with ''. - Evidence: if (token == '' || token != hex_md5(url + vandal + document.cookie)) {
36. ISSUE: line 595 character 11: Missing semicolon. - Evidence: return
37. ISSUE: line 605 character 11: Use '===' to compare with '0'. - Evidence: if (i == 0 && name != vandal) {
38. ISSUE: line 614 character 14: Use '===' to compare with ''. - Evidence: if (editor == '') {
39. ISSUE: line 630 character 11: Missing semicolon. - Evidence: return
40. ISSUE: line 682 character 29: Missing semicolon. - Evidence: var t = l[i].innerHTML
41. ISSUE: line 684 character 29: Use '!==' to compare with ''. - Evidence: if (n >= 0 && article != '' && t.indexOf('oldid=') == -1) {
42. ISSUE: line 700 character 13: 't' is already defined. - Evidence: var t = l[i].innerHTML
43. ISSUE: line 700 character 29: Missing semicolon. - Evidence: var t = l[i].innerHTML
44. ISSUE: line 728 character 1: document.write can be a form of eval. - Evidence: document.write('<script type="text/javascript" src="'
45. ISSUE: line 733 character 1: document.write can be a form of eval. - Evidence: document.write('<script type="text/javascript" src="'