Commons:User scripts/users namespace/reports/User:ABF/monobook.js
Jump to navigation
Jump to search
This is the report page for User:ABF/monobook.js. It is bot-maintained. All manual changes might be overwritten.
jshint[edit]
- ISSUE:
line 6 character 31
: The array literal notation [] is preferable. - Evidence:var UrlParameters = new Array ();
- ISSUE:
line 22 character 17
: ['action'] is better written in dot notation. - Evidence:if(UrlParameters["action"] == "edit" && UrlParameters["copyviotag"] == 1) {
- ISSUE:
line 22 character 54
: ['copyviotag'] is better written in dot notation. - Evidence:if(UrlParameters["action"] == "edit" && UrlParameters["copyviotag"] == 1) {
- ISSUE:
line 27 character 20
: eval can be harmful. - Evidence:var text = eval("(" + req.responseText + ")").query.pages;
- ISSUE:
line 32 character 23
: ['editform'] is better written in dot notation. - Evidence:document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
- ISSUE:
line 32 character 58
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
- ISSUE:
line 32 character 60
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
- ISSUE:
line 32 character 69
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
- ISSUE:
line 32 character 71
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += '\{\{copyvio\}\}';
- ISSUE:
line 33 character 23
: ['editform'] is better written in dot notation. - Evidence:document.forms["editform"].wpSummary.value = 'Tagging as a speedily deletable [[Commons:Licensing|copyright violation]] (using [[User:Foxj/copyvio.js|copyvio]] script)';
- ISSUE:
line 34 character 23
: ['editform'] is better written in dot notation. - Evidence:document.forms["editform"].wpMinoredit.checked = true;
- ISSUE:
line 35 character 23
: ['editform'] is better written in dot notation. - Evidence:document.forms["editform"].submit();
- ISSUE:
line 36 character 19
: 'user' used out of scope. - Evidence:if(typeof(user) != 'undefined') window.setTimeout(function() { location.href = wgScript + "?title=User_talk:" + user + "&action=edit©vionotify=1©vio=" + wgPageName; }, 1000);
- ISSUE:
line 40 character 17
: ['action'] is better written in dot notation. - Evidence:if(UrlParameters["action"] == "edit" && UrlParameters["copyvionotify"] == 1 && UrlParameters["copyvio"].length > 0 && wgNamespaceNumber == 3) {
- ISSUE:
line 40 character 54
: ['copyvionotify'] is better written in dot notation. - Evidence:if(UrlParameters["action"] == "edit" && UrlParameters["copyvionotify"] == 1 && UrlParameters["copyvio"].length > 0 && wgNamespaceNumber == 3) {
- ISSUE:
line 40 character 93
: ['copyvio'] is better written in dot notation. - Evidence:if(UrlParameters["action"] == "edit" && UrlParameters["copyvionotify"] == 1 && UrlParameters["copyvio"].length > 0 && wgNamespaceNumber == 3) {
- ISSUE:
line 42 character 55
: ['copyvio'] is better written in dot notation. - Evidence:var copyvio = decodeURIComponent(UrlParameters["copyvio"].replace(/_/g, " "));
- ISSUE:
line 43 character 23
: ['editform'] is better written in dot notation. - Evidence:document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
- ISSUE:
line 43 character 58
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
- ISSUE:
line 43 character 60
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
- ISSUE:
line 43 character 95
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
- ISSUE:
line 43 character 97
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
- ISSUE:
line 43 character 100
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
- ISSUE:
line 43 character 102
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
- ISSUE:
line 43 character 104
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
- ISSUE:
line 43 character 106
: Bad or unnecessary escaping. - Evidence:document.forms["editform"].wpTextbox1.value += "\{\{subst:copyvionote|" + copyvio + "\}\} \~\~\~\~";
- ISSUE:
line 44 character 23
: ['editform'] is better written in dot notation. - Evidence:document.forms["editform"].wpSummary.value = 'Notification - [[' + copyvio+ ']] is a probable [[Commons:Licensing|copyright violation]] (using [[User:Foxj/copyvio.js|copyvio]] script)';
- ISSUE:
line 45 character 23
: ['editform'] is better written in dot notation. - Evidence:document.forms["editform"].wpMinoredit.checked = true;
- ISSUE:
line 46 character 23
: ['editform'] is better written in dot notation. - Evidence:document.forms["editform"].submit();
- ISSUE:
line 51 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©viotag=1", "copyvio", "ca-copyvio");
- ISSUE:
line 111 character 16
: Use '!==' to compare with 'null'. - Evidence:while ((iter != null) && (count < 3)) {
- ISSUE:
line 112 character 38
: Use '===' to compare with '0'. - Evidence:if ((iter.nodeName.search(/div/i) == 0) && (iter.className.search(/usermessage/i) == 0)) {
- ISSUE:
line 112 character 86
: Use '===' to compare with '0'. - Evidence:if ((iter.nodeName.search(/div/i) == 0) && (iter.className.search(/usermessage/i) == 0)) {