Commons:User scripts/users namespace/reports/User:Aqwis/cvtagger.js

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

jshint

1. ISSUE: line 1 character 31: The array literal notation [] is preferable. - Evidence: var UrlParameters = new Array ();
2. ISSUE: line 17 character 17: ['action'] is better written in dot notation. - Evidence: if(UrlParameters["action"] == "edit" && UrlParameters["copyviotag"] == 1) {
3. ISSUE: line 17 character 54: ['copyviotag'] is better written in dot notation. - Evidence: if(UrlParameters["action"] == "edit" && UrlParameters["copyviotag"] == 1) {
4. ISSUE: line 22 character 20: eval can be harmful. - Evidence: var text = eval("(" + req.responseText + ")").query.pages;
5. ISSUE: line 27 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
6. ISSUE: line 27 character 58: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
7. ISSUE: line 27 character 60: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
8. ISSUE: line 27 character 69: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
9. ISSUE: line 27 character 71: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
10. ISSUE: line 28 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpSummary.value = 'Tagging as a [[Commons:Licensing|copyright violation]] (using [[User:Kanonkas/Copyviotagger.js|script]])';
11. ISSUE: line 29 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpMinoredit.checked = true;
12. ISSUE: line 30 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].submit();
13. ISSUE: line 31 character 19: 'user' used out of scope. - Evidence: if(typeof(user) != 'undefined') window.setTimeout(function() { location.href = wgScript + "?title=User_talk:" + user + "&action=edit&copyvionotify=1&copyvio=" + wgPageName; }, 1000);
14. ISSUE: line 35 character 17: ['action'] is better written in dot notation. - Evidence: if(UrlParameters["action"] == "edit" && UrlParameters["copyvionotify"] == 1 && UrlParameters["copyvio"].length > 0 && wgNamespaceNumber == 3) {
15. ISSUE: line 35 character 54: ['copyvionotify'] is better written in dot notation. - Evidence: if(UrlParameters["action"] == "edit" && UrlParameters["copyvionotify"] == 1 && UrlParameters["copyvio"].length > 0 && wgNamespaceNumber == 3) {
16. ISSUE: line 35 character 93: ['copyvio'] is better written in dot notation. - Evidence: if(UrlParameters["action"] == "edit" && UrlParameters["copyvionotify"] == 1 && UrlParameters["copyvio"].length > 0 && wgNamespaceNumber == 3) {
17. ISSUE: line 37 character 55: ['copyvio'] is better written in dot notation. - Evidence: var copyvio = decodeURIComponent(UrlParameters["copyvio"].replace(/_/g, " "));
18. ISSUE: line 38 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
19. ISSUE: line 38 character 58: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
20. ISSUE: line 38 character 60: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
21. ISSUE: line 38 character 95: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
22. ISSUE: line 38 character 97: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
23. ISSUE: line 38 character 100: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
24. ISSUE: line 38 character 102: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
25. ISSUE: line 38 character 104: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
26. ISSUE: line 38 character 106: Bad or unnecessary escaping. - Evidence: document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
27. ISSUE: line 39 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpSummary.value = 'Notification - [[' + copyvio+ ']] is a possible [[Commons:Licensing|copyright violation]] (using [[User:Kanonkas/Copyviotagger.js|script]])';
28. ISSUE: line 40 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpMinoredit.checked = true;
29. ISSUE: line 41 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].submit();
30. ISSUE: line 46 character 48: ['copyviotag'] is better written in dot notation. - Evidence: if(wgNamespaceNumber == 6 && !UrlParameters["copyviotag"]) mw.util.addPortletLink("p-cactions", wgScript + "?title=" + wgPageName + "&action=edit&copyviotag=1", "copyvio", "ca-copyvio");