Commons:User scripts/users namespace/reports/User:Lilyu/QuickDelete.js

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

jshint edit

  1. ISSUE: line 74 character 21: Bad or unnecessary escaping. - Evidence: tag: '{\{copyvio|1=%PARAMETER%}}',
  2. ISSUE: line 76 character 26: Bad or unnecessary escaping. - Evidence: talk_tag: '{\{subst:copyvionote|1=%FILE%}}',
  3. ISSUE: line 84 character 21: Bad or unnecessary escaping. - Evidence: tag: '{\{subst:nsd}}',
  4. ISSUE: line 86 character 26: Bad or unnecessary escaping. - Evidence: talk_tag: '{\{subst:image source|1=%FILE%}}',
  5. ISSUE: line 93 character 21: Bad or unnecessary escaping. - Evidence: tag: '{\{subst:npd}}',
  6. ISSUE: line 95 character 26: Bad or unnecessary escaping. - Evidence: talk_tag: '{\{subst:image permission|1=%FILE%}}',
  7. ISSUE: line 101 character 21: Bad or unnecessary escaping. - Evidence: tag: '{\{subst:nld}}',
  8. ISSUE: line 103 character 26: Bad or unnecessary escaping. - Evidence: talk_tag: '{\{subst:image license|1=%FILE%}}',
  9. ISSUE: line 120 character 78: Script URL. - Evidence: mw.util.addPortletLink('p-Gadgets', 'javascript:AQD2.nominateForDeletion();', this.i18n.toolboxLinkDelete, 't-AQD2', null);
  10. ISSUE: line 122 character 74: Script URL. - Evidence: mw.util.addPortletLink('p-Gadgets', 'javascript:AQD2.discussCategory();', this.i18n.toolboxLinkDiscuss, 't-AQD2discusscat', null);
  11. ISSUE: line 137 character 74: Script URL. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:AQD2.moveFile("", "");', this.i18n.dropdownMove, 'ca-quickmove', 'ca-move');
  12. ISSUE: line 154 character 76: Script URL. - Evidence: mw.util.addPortletLink('p-Gadgets', 'javascript:AQD2.insertTagOnPage("' + v.label + '","' + v.tag + '","' + v.ask + '","'
  13. ISSUE: line 269 character 21: Bad or unnecessary escaping. - Evidence: this.tag = '{\{subst:cfd}}';
  14. ISSUE: line 271 character 26: Bad or unnecessary escaping. - Evidence: this.talk_tag = "{\{subst:cdw|" + wgPageName + "}}";
  15. ISSUE: line 313 character 21: Bad or unnecessary escaping. - Evidence: this.tag = "{\{delete|reason=%PARAMETER%|subpage=" + this.pageName + this.formatDate("|year=YYYY|month=MON|day=DAY}}\n");
  16. ISSUE: line 315 character 26: Bad or unnecessary escaping. - Evidence: this.talk_tag = "{\{subst:idw|1=" + this.pageName + "}}"; //LILYU : adding 1= to clarify parameter and allow extraction of filename in notifyuploaders function
  17. ISSUE: line 423 character 62: Bad or unnecessary escaping. - Evidence: for (var user in this.uploaders) { page.text +="\n* {\{User|" + user +"|" + user +"}}" }
  18. ISSUE: line 444 character 24: Bad or unnecessary escaping. - Evidence: page.text = "\n{\{" + this.requestPage + "}}\n";
  19. ISSUE: line 460 character 54: Use '===' to compare with ''. - Evidence: if ( (user == wgUserName) || (this.talk_tag == '') ) continue; // notifying yourself is pointless
  20. ISSUE: line 475 character 48: Unnecessary semicolon. - Evidence: var filename = temp[1].split('}}')[0];;
  21. ISSUE: line 479 character 29: Bad or unnecessary escaping. - Evidence: page2.text = "\n*{\{User|"+user+"}} : [[:" + filename +"]] <small>\'\'\'"+this.label+"\'\'\'</small> ";
  22. ISSUE: line 523 character 20: 'query' is already defined. - Evidence: var query = {
  23. ISSUE: line 534 character 22: 'query' used out of scope. - Evidence: this.doAPICall(query, 'findCreatorCB');
  24. ISSUE: line 648 character 24: Bad or unnecessary escaping. - Evidence: page.text = '\n{\{universal replace|' + wgPageName.replace('File:', '') + '|'
  25. ISSUE: line 692 character 29: Use '!==' to compare with ''. - Evidence: if (response != '')
  26. ISSUE: line 701 character 37: Use '!==' to compare with ''. - Evidence: if (AQD2.talk_tag != '')
  27. ISSUE: line 756 character 58: Use '!==' to compare with 'false'. - Evidence: if (event.keyCode == '13' && v.enterToSubmit != false) submitButton.click();
  28. ISSUE: line 843 character 38: Use '!==' to compare with 'null'. - Evidence: while ((m = utf8.exec(str)) != null) {
  29. ISSUE: line 1119 character 73: Bad or unnecessary escaping. - Evidence: prependDeletionTemplate: "An error occurred while adding the {\{delete}} template to this "
  30. ISSUE: line 1127 character 69: Bad or unnecessary escaping. - Evidence: + " for deletion, please edit the page to add the {\{delete}} template and follow the instructions shown on it.",