Commons:User scripts/users namespace/reports/User:Satnav/Gadget-Cat-a-lot.js

This is the report page for User:Satnav/Gadget-Cat-a-lot.js. It is bot-maintained. All manual changes might be overwritten.

jshint

1. ISSUE: line 81 character 40: The array literal notation [] is preferable. - Evidence: catALot.subCats = new Array();
2. ISSUE: line 98 character 43: The array literal notation [] is preferable. - Evidence: catALot.parentCats = new Array();
3. ISSUE: line 100 character 48: Use '===' to compare with ''. - Evidence: if (pages[-1] && pages[-1].missing == '') {
4. ISSUE: line 112 character 33: 'cats' used out of scope. - Evidence: for (var i = 0; i < cats.length; i++) {
5. ISSUE: line 113 character 41: 'cats' used out of scope. - Evidence: catALot.parentCats.push(cats[i].title.replace(/^[^:]+:/, ""));
6. ISSUE: line 165 character 20: Use '===' to compare with 'null'. - Evidence: if (result == null) {
7. ISSUE: line 179 character 51: ['timestamp'] is better written in dot notation. - Evidence: var timestamp = pages[id].revisions[0]['timestamp'];
8. ISSUE: line 185 character 121: 'otext' used out of scope. - Evidence: if (mode != "remove" && sourcecat.toLowerCase() != targetcat.toLowerCase() && this.regexBuilder(targetcat).test(otext)) {
9. ISSUE: line 197 character 20: 'otext' used out of scope. - Evidence: var text = otext;
10. ISSUE: line 210 character 18: 'otext' used out of scope. - Evidence: if ( otext == text ) {
11. ISSUE: line 224 character 21: 'otext' used out of scope. - Evidence: if (text == otext) {
12. ISSUE: line 240 character 29: 'starttimestamp' used out of scope. - Evidence: starttimestamp: starttimestamp,
13. ISSUE: line 241 character 28: 'timestamp' used out of scope. - Evidence: basetimestamp: timestamp,
14. ISSUE: line 322 character 26: Use '===' to compare with '0'. - Evidence: if (files.length == 0) {
15. ISSUE: line 353 character 10: Missing semicolon. - Evidence: }
16. ISSUE: line 379 character 14: Don't make functions within a loop. - Evidence: });
17. ISSUE: line 385 character 18: Don't make functions within a loop. - Evidence: });
18. ISSUE: line 390 character 18: Don't make functions within a loop. - Evidence: });
19. ISSUE: line 395 character 18: Don't make functions within a loop. - Evidence: });
20. ISSUE: line 401 character 78: 'add' used out of scope. - Evidence: if (list[i] != wgTitle && this.searchmode) li.append(' ').append(add);
21. ISSUE: line 402 character 84: 'move' used out of scope. - Evidence: else if (list[i] != wgTitle && !this.searchmode) li.append(' ').append(move).append(' ').append(copy);
22. ISSUE: line 402 character 109: 'copy' used out of scope. - Evidence: else if (list[i] != wgTitle && !this.searchmode) li.append(' ').append(move).append(' ').append(copy);