Commons:User scripts/users namespace/reports/User:Animum/moveRV-severe.js

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

jshint

1. ISSUE: line 27 character 51: eval can be harmful. - Evidence: var pageInfo = formatResponse(eval("(" + req.responseText + ")"));
2. ISSUE: line 28 character 43: Use '!==' to compare with 'undefined'. - Evidence: if(pageInfo.revisions != undefined) {
3. ISSUE: line 33 character 29: 'shouldAbort' used out of scope. - Evidence: shouldAbort = false;
4. ISSUE: line 36 character 25: 'shouldAbort' used out of scope. - Evidence: shouldAbort = false; //No comment means no move has been made.
5. ISSUE: line 39 character 25: 'shouldAbort' used out of scope. - Evidence: shouldAbort = false; //If page doesn't exist, we don't want to move it.
6. ISSUE: line 41 character 33: 'req' is already defined. - Evidence: var req = sajax_init_object();
7. ISSUE: line 44 character 42: 'pageInfo' is already defined. - Evidence: var pageInfo = formatResponse(eval("(" + req.responseText + ")"));
8. ISSUE: line 44 character 59: eval can be harmful. - Evidence: var pageInfo = formatResponse(eval("(" + req.responseText + ")"));
9. ISSUE: line 45 character 51: Use '===' to compare with 'undefined'. - Evidence: if(pageInfo.revisions == undefined) {
10. ISSUE: line 46 character 34: 'shouldAbort' used out of scope. - Evidence: shouldAbort = true; //Page to move doesn't exist.
11. ISSUE: line 48 character 33: 'req' is already defined. - Evidence: var req = sajax_init_object();
12. ISSUE: line 52 character 55: eval can be harmful. - Evidence: var info = formatResponse(eval("(" + req.responseText + ")"));
13. ISSUE: line 62 character 33: 'req' is already defined. - Evidence: var req = sajax_init_object();
14. ISSUE: line 66 character 28: 'shouldAbort' used out of scope. - Evidence: if(shouldAbort == false) { //If nobody else has already reverted, we do
15. ISSUE: line 66 character 40: Use '===' to compare with 'false'. - Evidence: if(shouldAbort == false) { //If nobody else has already reverted, we do
16. ISSUE: line 76 character 37: 'req' is already defined. - Evidence: var req = sajax_init_object();
17. ISSUE: line 79 character 32: eval can be harmful. - Evidence: if(eval("(" + req.responseText + ")").query.logevents) {
18. ISSUE: line 80 character 36: eval can be harmful. - Evidence: if(eval("(" + req.responseText + ")").query.logevents[0].user == wgUserName && eval("(" + req.responseText + ")").query.logevents[0].comment.indexOf(summary) != -1) {
19. ISSUE: line 80 character 112: eval can be harmful. - Evidence: if(eval("(" + req.responseText + ")").query.logevents[0].user == wgUserName && eval("(" + req.responseText + ")").query.logevents[0].comment.indexOf(summary) != -1) {
20. ISSUE: line 84 character 54: 'postdata' is already defined. - Evidence: var postdata = "wpReason=" + encodeURIComponent(delsummary)
21. ISSUE: line 86 character 49: 'req' is already defined. - Evidence: var req = sajax_init_object();
22. ISSUE: line 109 character 29: eval can be harmful. - Evidence: var info = formatResponse(eval("(" + req.responseText + ")"));
23. ISSUE: line 126 character 16: eval can be harmful. - Evidence: var info = eval("(" + req.responseText + ")");
24. ISSUE: line 140 character 29: eval can be harmful. - Evidence: var info = formatResponse(eval("(" + req.responseText + ")"));
25. ISSUE: line 148 character 11: 'req' is already defined. - Evidence: var req = sajax_init_object();
26. ISSUE: line 164 character 14: eval can be harmful. - Evidence: var info = eval("(" + req.responseText + ")");
27. ISSUE: line 195 character 11: 'req' is already defined. - Evidence: var req = sajax_init_object();
28. ISSUE: line 207 character 80: ['user'] is better written in dot notation. - Evidence: var doContinue = confirm("This script will block " + unescape(UrlParameters["user"].replace(/[+_]/g, " ")) + ", protect his/her talk page, revert ALL of his/her moves, and delete the redirects created therefrom.\n\nClick \"OK\" to continue, or \"Cancel\" to abort.");
29. ISSUE: line 209 character 39: ['user'] is better written in dot notation. - Evidence: blockip(unescape(UrlParameters["user"]).replace(/[+_]/g, " "));
30. ISSUE: line 216 character 88: ['type'] is better written in dot notation. - Evidence: if(wgNamespaceNumber == -1 && wgCanonicalSpecialPageName == "Log" && (UrlParameters["type"] == "move" || location.href.substring(location.href.indexOf("title=") + 6).split("/")[1] == "move" || location.href.substring(location.href.indexOf("/wiki/") + 6).split("?")[0].split("/")[1] == "move") && (UrlParameters["user"] || location.href.substring(location.href.indexOf("user=") + 5))) {
31. ISSUE: line 216 character 315: ['user'] is better written in dot notation. - Evidence: if(wgNamespaceNumber == -1 && wgCanonicalSpecialPageName == "Log" && (UrlParameters["type"] == "move" || location.href.substring(location.href.indexOf("title=") + 6).split("/")[1] == "move" || location.href.substring(location.href.indexOf("/wiki/") + 6).split("?")[0].split("/")[1] == "move") && (UrlParameters["user"] || location.href.substring(location.href.indexOf("user=") + 5))) {
32. ISSUE: line 217 character 71: Script URL. - Evidence: mw.util.addPortletLink("p-cactions", "javascript:beginRectification()", "pagemove", "ca-pagemove");