Commons:User scripts/users namespace/reports/User:ThiagoRuiz/common.js

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

jshint

1. ISSUE: line 95 character 40: The array literal notation [] is preferable. - Evidence: catALot.subCats = new Array();
2. ISSUE: line 112 character 43: The array literal notation [] is preferable. - Evidence: catALot.parentCats = new Array();
3. ISSUE: line 114 character 48: Use '===' to compare with ''. - Evidence: if (pages[-1] && pages[-1].missing == '') {
4. ISSUE: line 126 character 33: 'cats' used out of scope. - Evidence: for (var i = 0; i < cats.length; i++) {
5. ISSUE: line 127 character 41: 'cats' used out of scope. - Evidence: catALot.parentCats.push(cats[i].title.replace(/^[^:]+:/, ""));
6. ISSUE: line 174 character 20: Use '===' to compare with 'null'. - Evidence: if (result == null) {
7. ISSUE: line 188 character 51: ['timestamp'] is better written in dot notation. - Evidence: var timestamp = pages[id].revisions[0]['timestamp'];
8. ISSUE: line 195 character 67: 'otext' used out of scope. - Evidence: if (mode != "remove" && this.regexBuilder(targetcat).test(otext)) {
9. ISSUE: line 206 character 20: 'otext' used out of scope. - Evidence: var text = otext;
10. ISSUE: line 229 character 21: 'otext' used out of scope. - Evidence: if (text == otext) {
11. ISSUE: line 240 character 29: 'starttimestamp' used out of scope. - Evidence: starttimestamp: starttimestamp,
12. ISSUE: line 241 character 28: 'timestamp' used out of scope. - Evidence: basetimestamp: timestamp,
13. ISSUE: line 322 character 26: Use '===' to compare with '0'. - Evidence: if (files.length == 0) {
14. ISSUE: line 361 character 14: Don't make functions within a loop. - Evidence: });
15. ISSUE: line 367 character 18: Don't make functions within a loop. - Evidence: });
16. ISSUE: line 372 character 18: Don't make functions within a loop. - Evidence: });
17. ISSUE: line 377 character 18: Don't make functions within a loop. - Evidence: });
18. ISSUE: line 379 character 25: 'add' is already defined. - Evidence: var add = $('<a class="cat_a_lot_action"><b>' + this.i18n.add + '</b></a>');
19. ISSUE: line 382 character 18: Don't make functions within a loop. - Evidence: });
20. ISSUE: line 388 character 78: 'add' used out of scope. - Evidence: if (list[i] != wgTitle && this.searchmode) li.append(' ').append(add);
21. ISSUE: line 389 character 84: 'move' used out of scope. - Evidence: else if (list[i] != wgTitle && !this.searchmode) li.append(' ').append(move).append(' ').append(add);
22. ISSUE: line 389 character 109: 'add' used out of scope. - Evidence: else if (list[i] != wgTitle && !this.searchmode) li.append(' ').append(move).append(' ').append(add);
23. ISSUE: line 561 character 62: Bad or unnecessary escaping. - Evidence: regex(/[dD]escription[ ]*\=[ ]*\{\{/,'Description=\n\{\{'); // line break after description=
24. ISSUE: line 561 character 64: Bad or unnecessary escaping. - Evidence: regex(/[dD]escription[ ]*\=[ ]*\{\{/,'Description=\n\{\{'); // line break after description=
25. ISSUE: line 563 character 15: Unexpected escaped character '<' in regular expression. - Evidence: regex(/\<\!--\{\{ImageUpload\|full\}\}--\>[\s\S]/,''); // remove comment