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

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

jshint

1. ISSUE: line 17 character 37: Missing semicolon. - Evidence: popupFixDabsSummary='BKL aufgelöst:'
2. ISSUE: line 20 character 1: document.write can be a form of eval. - Evidence: document.write('<script type="text/javascript" src="/w/index.php?title=' +
3. ISSUE: line 25 character 1: document.write can be a form of eval. - Evidence: document.write('<script type="text/javascript" src="'
4. ISSUE: line 64 character 69: Use '===' to compare with 'null'. - Evidence: if ( wgNamespaceNumber != -1 || wgTitle != "Upload" || wgUserName == null) return ;
5. ISSUE: line 68 character 10: Use '===' to compare with 'null'. - Evidence: if (ip == null) return;
6. ISSUE: line 87 character 13: Use '===' to compare with 'null'. - Evidence: if (label == null)
7. ISSUE: line 127 character 13: Use '!==' to compare with 'null'. - Evidence: if (input != null) hotcat_ok ();
8. ISSUE: line 133 character 19: eval can be harmful. - Evidence: do_submit = eval (hotcat_old_onsubmit);
9. ISSUE: line 146 character 21: Bad or unnecessary escaping. - Evidence: var new_cat = "\[\[Category:" + t + "\]\]" ;
10. ISSUE: line 146 character 23: Bad or unnecessary escaping. - Evidence: var new_cat = "\[\[Category:" + t + "\]\]" ;
11. ISSUE: line 146 character 43: Bad or unnecessary escaping. - Evidence: var new_cat = "\[\[Category:" + t + "\]\]" ;
12. ISSUE: line 146 character 45: Bad or unnecessary escaping. - Evidence: var new_cat = "\[\[Category:" + t + "\]\]" ;
13. ISSUE: line 161 character 52: Use '!==' to compare with 'null'. - Evidence: if( document.getElementById('ca-viewsource' ) != null // User has no permission to edit
14. ISSUE: line 175 character 20: Use '===' to compare with 'null'. - Evidence: if (visible_cats == null) {
15. ISSUE: line 178 character 21: Use '===' to compare with 'null'. - Evidence: if (hidden_cats == null) {
16. ISSUE: line 190 character 21: Use '===' to compare with 'null'. - Evidence: if (hidden_cats == null) {
17. ISSUE: line 210 character 43: Use '!==' to compare with 'null'. - Evidence: && UploadForm.previous_hotcat_state != null)
18. ISSUE: line 226 character 2: Missing semicolon. - Evidence: }
19. ISSUE: line 275 character 51: Script URL. - Evidence: modify_link.href = "javascript:hotcat_modify(\"" + mod_id + "\");" ;
20. ISSUE: line 378 character 26: The array literal notation [] is preferable. - Evidence: var result = new Array ();
21. ISSUE: line 380 character 51: Use '!==' to compare with 'null'. - Evidence: while ((curr_match = cat_regex.exec (wikitext)) != null) {
22. ISSUE: line 394 character 27: The array literal notation [] is preferable. - Evidence: var summary = new Array () ;
23. ISSUE: line 406 character 14: Use '!==' to compare with 'null'. - Evidence: if (cat_rm != null && cat_rm.length > 0) {
24. ISSUE: line 408 character 36: Use '===' to compare with '0'. - Evidence: if (!matches || matches.length == 0) {
25. ISSUE: line 416 character 19: Use '!==' to compare with 'null'. - Evidence: if (cat_add != null && cat_add.length > 0 && matches[0].match.length > 1)
26. ISSUE: line 439 character 15: Use '!==' to compare with 'null'. - Evidence: if (cat_add != null && cat_add.length > 0) {
27. ISSUE: line 440 character 17: 'matches' is already defined. - Evidence: var matches = hotcat_find_category (t, cat_add);
28. ISSUE: line 446 character 17: Bad or unnecessary escaping. - Evidence: t = t + '\[\[Category:' + cat_add + (cat_key != null ? cat_key : "") + '\]\]\n';
29. ISSUE: line 446 character 19: Bad or unnecessary escaping. - Evidence: t = t + '\[\[Category:' + cat_add + (cat_key != null ? cat_key : "") + '\]\]\n';
30. ISSUE: line 446 character 52: Use '!==' to compare with 'null'. - Evidence: t = t + '\[\[Category:' + cat_add + (cat_key != null ? cat_key : "") + '\]\]\n';
31. ISSUE: line 446 character 80: Bad or unnecessary escaping. - Evidence: t = t + '\[\[Category:' + cat_add + (cat_key != null ? cat_key : "") + '\]\]\n';
32. ISSUE: line 446 character 82: Bad or unnecessary escaping. - Evidence: t = t + '\[\[Category:' + cat_add + (cat_key != null ? cat_key : "") + '\]\]\n';
33. ISSUE: line 448 character 14: 't2' is already defined. - Evidence: var t2 = t.replace(hotcat_uncat_regex, ""); // Remove "uncat" templates
34. ISSUE: line 482 character 45: Script URL. - Evidence: a_add.href = "javascript:hotcat_add_new()" ;
35. ISSUE: line 520 character 13: Use '===' to compare with 'null'. - Evidence: if (label == null || !label.data) return defaultText;
36. ISSUE: line 540 character 15: Use '!==' to compare with 'null'. - Evidence: if (l != null)
37. ISSUE: line 547 character 15: Use '!==' to compare with 'null'. - Evidence: if (l != null)
38. ISSUE: line 564 character 89: ['HotCatDelay'] is better written in dot notation. - Evidence: text.onkeyup = function () { window.setTimeout("hotcat_text_changed();", JSconfig.keys['HotCatDelay'] ); } ;
39. ISSUE: line 564 character 38: Implied eval. Consider passing a function instead of a string. - Evidence: text.onkeyup = function () { window.setTimeout("hotcat_text_changed();", JSconfig.keys['HotCatDelay'] ); } ;
40. ISSUE: line 583 character 12: Use '!==' to compare with 'null'. - Evidence: if (list != null) form.appendChild ( list ) ;
41. ISSUE: line 585 character 14: Use '!==' to compare with 'null'. - Evidence: if (exists != null) form.appendChild ( exists ) ;
42. ISSUE: line 608 character 15: Use '===' to compare with 'null'. - Evidence: if (request == null) {
43. ISSUE: line 622 character 25: eval can be harmful. - Evidence: var do_submit = eval (request.responseText);
44. ISSUE: line 627 character 53: Bad or unnecessary escaping. - Evidence: ,(txt && txt.value != v) ? " (redirect \[\[:Category:" + v + "|" + v + "\]\] resolved)" : null
45. ISSUE: line 627 character 55: Bad or unnecessary escaping. - Evidence: ,(txt && txt.value != v) ? " (redirect \[\[:Category:" + v + "|" + v + "\]\] resolved)" : null
46. ISSUE: line 627 character 86: Bad or unnecessary escaping. - Evidence: ,(txt && txt.value != v) ? " (redirect \[\[:Category:" + v + "|" + v + "\]\] resolved)" : null
47. ISSUE: line 627 character 88: Bad or unnecessary escaping. - Evidence: ,(txt && txt.value != v) ? " (redirect \[\[:Category:" + v + "|" + v + "\]\] resolved)" : null
48. ISSUE: line 646 character 26: ['title'] is better written in dot notation. - Evidence: var cat = cats[c]["title"];
49. ISSUE: line 657 character 28: The array literal notation [] is preferable. - Evidence: var titles = new Array ();
50. ISSUE: line 659 character 20: ['ns'] is better written in dot notation. - Evidence: if ( lks[i]["ns"] == 14 // Category namespace
51. ISSUE: line 659 character 20: Too many errors. (68% scanned). - Evidence: undefined