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

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

jshint

1. ISSUE: line 24 character 51: eval can be harmful. - Evidence: var pageInfo = formatResponse(eval("(" + req.responseText + ")"));
2. ISSUE: line 25 character 43: Use '!==' to compare with 'undefined'. - Evidence: if(pageInfo.revisions != undefined) {
3. ISSUE: line 30 character 29: 'shouldAbort' used out of scope. - Evidence: shouldAbort = false;
4. ISSUE: line 33 character 25: 'shouldAbort' used out of scope. - Evidence: shouldAbort = false; //No comment means no move has been made.
5. ISSUE: line 36 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 38 character 33: 'req' is already defined. - Evidence: var req = sajax_init_object();
7. ISSUE: line 41 character 42: 'pageInfo' is already defined. - Evidence: var pageInfo = formatResponse(eval("(" + req.responseText + ")"));
8. ISSUE: line 41 character 59: eval can be harmful. - Evidence: var pageInfo = formatResponse(eval("(" + req.responseText + ")"));
9. ISSUE: line 42 character 51: Use '===' to compare with 'undefined'. - Evidence: if(pageInfo.revisions == undefined) {
10. ISSUE: line 43 character 34: 'shouldAbort' used out of scope. - Evidence: shouldAbort = true; //Page to move doesn't exist.
11. ISSUE: line 45 character 33: 'req' is already defined. - Evidence: var req = sajax_init_object();
12. ISSUE: line 49 character 55: eval can be harmful. - Evidence: var info = formatResponse(eval("(" + req.responseText + ")"));
13. ISSUE: line 59 character 33: 'req' is already defined. - Evidence: var req = sajax_init_object();
14. ISSUE: line 63 character 28: 'shouldAbort' used out of scope. - Evidence: if(shouldAbort == false) { //If nobody else has already reverted, we do
15. ISSUE: line 63 character 40: Use '===' to compare with 'false'. - Evidence: if(shouldAbort == false) { //If nobody else has already reverted, we do
16. ISSUE: line 73 character 37: 'req' is already defined. - Evidence: var req = sajax_init_object();
17. ISSUE: line 76 character 32: eval can be harmful. - Evidence: if(eval("(" + req.responseText + ")").query.logevents[0]) {
18. ISSUE: line 77 character 36: eval can be harmful. - Evidence: if(eval("(" + req.responseText + ")").query.logevents[0].user == wgUserName && eval("(" + req.responseText + ")").query.logevents[0].comment == summary) {
19. ISSUE: line 77 character 112: eval can be harmful. - Evidence: if(eval("(" + req.responseText + ")").query.logevents[0].user == wgUserName && eval("(" + req.responseText + ")").query.logevents[0].comment == summary) {
20. ISSUE: line 81 character 54: 'postdata' is already defined. - Evidence: var postdata = "wpReason=" + encodeURIComponent(delsummary)
21. ISSUE: line 83 character 49: 'req' is already defined. - Evidence: var req = sajax_init_object();
22. ISSUE: line 112 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))) {
23. ISSUE: line 112 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))) {
24. ISSUE: line 113 character 61: Script URL. - Evidence: mw.util.addPortletLink("p-cactions", "javascript:doMoveRV()", "pagemove", "ca-pagemove");