Commons:User scripts/users namespace/reports/User:Ianezz/Catwatch2.js

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

jshint

1. ISSUE: line 70 character 6: Missing semicolon. - Evidence: }
2. ISSUE: line 89 character 6: Missing semicolon. - Evidence: }
3. ISSUE: line 100 character 6: Missing semicolon. - Evidence: }
4. ISSUE: line 111 character 37: Use '===' to compare with 'undefined'. - Evidence: return !(this.metadata[key] == undefined);
5. ISSUE: line 111 character 16: Confusing use of '!'. - Evidence: return !(this.metadata[key] == undefined);
6. ISSUE: line 112 character 6: Missing semicolon. - Evidence: }
7. ISSUE: line 138 character 40: Use '===' to compare with '0'. - Evidence: if (this.prop.lowwatermark == 0) {
8. ISSUE: line 157 character 32: Use '===' to compare with '0'. - Evidence: if ( x == 0 ) return a.hits - b.hits;
9. ISSUE: line 166 character 32: Use '===' to compare with '0'. - Evidence: if ( x == 0 ) return a.ts - b.ts;
10. ISSUE: line 185 character 6: Missing semicolon. - Evidence: }
11. ISSUE: line 186 character 2: Unnecessary semicolon. - Evidence: };
12. ISSUE: line 229 character 26: The array literal notation [] is preferable. - Evidence: var p = new Array();
13. ISSUE: line 241 character 32: Use '!==' to compare with ''. - Evidence: if (baseURL && baseURL != "") {
14. ISSUE: line 267 character 27: Missing semicolon. - Evidence: f = JSON.parse
15. ISSUE: line 272 character 25: eval can be harmful. - Evidence: return (eval("(" + d + ")"));
16. ISSUE: line 273 character 14: Missing semicolon. - Evidence: }
17. ISSUE: line 309 character 78: Unnecessary semicolon. - Evidence: var i; for(i in queryParams) { queryParamsCopy[i] = queryParams[i]; };
18. ISSUE: line 412 character 25: Creating global 'for' variable. Should be 'for (var j ...'. - Evidence: for(j in obj["query-continue"]) {
19. ISSUE: line 443 character 41: 'queryParamsCopy' is already defined. - Evidence: var queryParamsCopy = {};
20. ISSUE: line 444 character 26: 'i' is already defined. - Evidence: var i; for (i in queryParams) { queryParamsCopy[i] = queryParams[i]; };
21. ISSUE: line 444 character 91: Unnecessary semicolon. - Evidence: var i; for (i in queryParams) { queryParamsCopy[i] = queryParams[i]; };
22. ISSUE: line 445 character 73: Unnecessary semicolon. - Evidence: var j; for (j in qc) { queryParamsCopy[j] = qc[j]; };
23. ISSUE: line 449 character 17: 'thereIsMoreWorkToDo' used out of scope. - Evidence: } while(thereIsMoreWorkToDo);
24. ISSUE: line 493 character 76: Unnecessary semicolon. - Evidence: var i; for(i in parameters) { queryParamsCopy[i] = parameters[i]; };
25. ISSUE: line 553 character 25: Creating global 'for' variable. Should be 'for (var j ...'. - Evidence: for(j in resultobj["query-continue"]) {
26. ISSUE: line 585 character 89: Unnecessary semicolon. - Evidence: var i; for (i in parameters) { queryParamsCopy[i] = parameters[i]; };
27. ISSUE: line 586 character 73: Unnecessary semicolon. - Evidence: var j; for (j in qc) { queryParamsCopy[j] = qc[j]; };
28. ISSUE: line 595 character 14: Missing semicolon. - Evidence: }
29. ISSUE: line 625 character 18: Missing semicolon. - Evidence: }
30. ISSUE: line 649 character 26: 'that' is already defined. - Evidence: var that = this;
31. ISSUE: line 653 character 33: 'thisrequest' is already defined. - Evidence: var thisrequest = request;
32. ISSUE: line 662 character 18: Missing semicolon. - Evidence: }
33. ISSUE: line 672 character 17: Expected a 'break' statement before 'default'. - Evidence: }
34. ISSUE: line 697 character 19: Use '===' to compare with 'undefined'. - Evidence: if (date1 == undefined || date2 == undefined) return false;
35. ISSUE: line 697 character 41: Use '===' to compare with 'undefined'. - Evidence: if (date1 == undefined || date2 == undefined) return false;
36. ISSUE: line 717 character 23: Use '===' to compare with 'undefined'. - Evidence: if (padlength == undefined) {padlength = 2}
37. ISSUE: line 718 character 23: Use '===' to compare with 'undefined'. - Evidence: if (padchar == undefined) {padchar="0"}
38. ISSUE: line 857 character 42: Missing semicolon. - Evidence: return { regexp: re, prefix: prefix }
39. ISSUE: line 896 character 14: Missing semicolon. - Evidence: }
40. ISSUE: line 945 character 50: Use '===' to compare with '0'. - Evidence: if(category.indexOf(prefixes[p]+":") == 0) {
41. ISSUE: line 990 character 18: Use '===' to compare with 'undefined'. - Evidence: if(label == undefined || label == null || label == "") {
42. ISSUE: line 990 character 40: Use '===' to compare with 'null'. - Evidence: if(label == undefined || label == null || label == "") {
43. ISSUE: line 990 character 57: Use '===' to compare with ''. - Evidence: if(label == undefined || label == null || label == "") {
44. ISSUE: line 1000 character 22: Use '!==' to compare with ''. - Evidence: if(pageTitle != "") {
45. ISSUE: line 1029 character 40: Use '===' to compare with '0'. - Evidence: if (page.revisions[0].parentid == 0) {
46. ISSUE: line 1096 character 25: Unnecessary semicolon. - Evidence: } catch (err) {};
47. ISSUE: line 1239 character 52: Missing semicolon. - Evidence: staticdata.pendingrequests += categories.length
48. ISSUE: line 1251 character 14: Missing semicolon. - Evidence: }
49. ISSUE: line 1281 character 57: Use '===' to compare with '0'. - Evidence: page.cw2isnew = (page.revisions[0].parentid == 0);
50. ISSUE: line 1282 character 58: Use '===' to compare with 'undefined'. - Evidence: page.cw2isminor = ! (page.revisions[0].minor == undefined);
51. ISSUE: line 1282 character 58: Too many errors. (68% scanned). - Evidence: undefined