Commons:User scripts/users namespace/reports/User:Fetchcomms/delete.js

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

jshint

1. ISSUE: line 44 character 93: Script URL. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:AjaxQuickDelete.nominateForDeletion();',
2. ISSUE: line 47 character 90: Script URL. - Evidence: addToolLink('p-cactions', 'javascript:AjaxQuickDelete.nominateForDeletion();',
3. ISSUE: line 59 character 31: Bad or unnecessary escaping. - Evidence: 'tag': '{\{copyvio|1=%PARAMETER%}}',
4. ISSUE: line 60 character 36: Bad or unnecessary escaping. - Evidence: 'talk_tag': '{\{subst:copyvionote|1=%FILE%}}',
5. ISSUE: line 68 character 31: Bad or unnecessary escaping. - Evidence: 'tag': '{\{subst:nsd}}',
6. ISSUE: line 69 character 36: Bad or unnecessary escaping. - Evidence: 'talk_tag': '{\{subst:image source|1=%FILE%}}',
7. ISSUE: line 77 character 31: Bad or unnecessary escaping. - Evidence: 'tag': '{\{subst:npd}}',
8. ISSUE: line 78 character 36: Bad or unnecessary escaping. - Evidence: 'talk_tag': '{\{subst:image permission|1=%FILE%}}',
9. ISSUE: line 85 character 31: Bad or unnecessary escaping. - Evidence: 'tag': '{\{subst:nld}}',
10. ISSUE: line 86 character 36: Bad or unnecessary escaping. - Evidence: 'talk_tag': '{\{subst:image license|1=%FILE%}}',
11. ISSUE: line 95 character 100: Script URL. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
12. ISSUE: line 95 character 106: ['tag'] is better written in dot notation. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
13. ISSUE: line 95 character 127: ['img_summary'] is better written in dot notation. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
14. ISSUE: line 95 character 156: ['talk_tag'] is better written in dot notation. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
15. ISSUE: line 95 character 182: ['talk_summary'] is better written in dot notation. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
16. ISSUE: line 95 character 212: ['prompt_text'] is better written in dot notation. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
17. ISSUE: line 95 character 240: ['label'] is better written in dot notation. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
18. ISSUE: line 98 character 97: Script URL. - Evidence: addToolLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
19. ISSUE: line 98 character 103: ['tag'] is better written in dot notation. - Evidence: addToolLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
20. ISSUE: line 98 character 124: ['img_summary'] is better written in dot notation. - Evidence: addToolLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
21. ISSUE: line 98 character 153: ['talk_tag'] is better written in dot notation. - Evidence: addToolLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
22. ISSUE: line 98 character 179: ['talk_summary'] is better written in dot notation. - Evidence: addToolLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
23. ISSUE: line 98 character 209: ['prompt_text'] is better written in dot notation. - Evidence: addToolLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
24. ISSUE: line 98 character 237: ['label'] is better written in dot notation. - Evidence: addToolLink('p-cactions', 'javascript:AjaxQuickDelete.insertTagOnPage("' + inb['tag'] + '","' + inb['img_summary'] + '","' + inb['talk_tag'] + '","' + inb['talk_summary'] + '","' + inb['prompt_text'] + '");', inb['label']);
25. ISSUE: line 302 character 26: 'page' is already defined. - Evidence: var page = [];
26. ISSUE: line 318 character 36: Use '===' to compare with '0'. - Evidence: if (this.uploadersToNotify == 0) this.nextTask();
27. ISSUE: line 323 character 36: Use '===' to compare with '0'. - Evidence: if (this.uploadersToNotify == 0) this.nextTask();
28. ISSUE: line 426 character 47: ['from'] is better written in dot notation. - Evidence: delete this.uploaders[redirect['from'].replace(this.userTalkPrefix, '')];
29. ISSUE: line 428 character 41: ['to'] is better written in dot notation. - Evidence: this.uploaders[ redirect['to'].replace(this.userTalkPrefix, '') ] = true;
30. ISSUE: line 470 character 30: eval can be harmful. - Evidence: var result = eval( "(" + x.responseText + ")" );
31. ISSUE: line 474 character 18: 'result' used out of scope. - Evidence: if (!result) return o.fail( "Receive empty API response:\n" + x.responseText );
32. ISSUE: line 475 character 17: 'result' used out of scope. - Evidence: if (result.error) return o.fail( "API request failed (" + result.error.code + "): " + result.error.info );
33. ISSUE: line 475 character 71: 'result' used out of scope. - Evidence: if (result.error) return o.fail( "API request failed (" + result.error.code + "): " + result.error.info );
34. ISSUE: line 475 character 99: 'result' used out of scope. - Evidence: if (result.error) return o.fail( "API request failed (" + result.error.code + "): " + result.error.info );
35. ISSUE: line 476 character 31: 'result' used out of scope. - Evidence: try { o[callback](result); } catch (e) { return o.fail(e); }
36. ISSUE: line 518 character 38: Use '!==' to compare with 'null'. - Evidence: while ((m = utf8.exec (str)) != null) {