Commons:User scripts/users namespace/reports/User:Kanonkas/batchdelete.js
This is the report page for User:Kanonkas/batchdelete.js. It is bot-maintained. All manual changes might be overwritten.
jshint edit
- ISSUE:
line 36 character 53
: Missing semicolon. - Evidence:while(fh.firstChild) fh.removeChild(fh.firstChild)
- ISSUE:
line 41 character 63
: Missing semicolon. - Evidence:while(bcur.firstChild) bcur.removeChild(bcur.firstChild)
- ISSUE:
line 50 character 52
: Script URL. - Evidence:form.setAttribute('action','javascript:void(0);');
- ISSUE:
line 58 character 42
: Missing semicolon. - Evidence:lab1.setAttribute('for','abd-reason')
- ISSUE:
line 85 character 11
: Use '===' to compare with ''. - Evidence:if(page == '') {
- ISSUE:
line 89 character 20
: Unexpected escaped character '<' in regular expression. - Evidence:var badchars = /(\#|\<|\>|\[|\]|\{|\}|\|)/;
- ISSUE:
line 92 character 7
: Implied eval. Consider passing a function instead of a string. - Evidence:setTimeout('abdStart()',1000);
- ISSUE:
line 110 character 7
: eval can be harmful. - Evidence:eval("abdDelete(" + req.responseText + ",'" + req.responseText.replace(/\'/g,"`") + "','" + page + "')");
- ISSUE:
line 112 character 4
: Missing semicolon. - Evidence:}
- ISSUE:
line 118 character 9
: ['error'] is better written in dot notation. - Evidence:if(obj['error']) {
- ISSUE:
line 119 character 67
: ['error'] is better written in dot notation. - Evidence:out.appendChild(document.createTextNode(' ! Api error: ' + obj['error']['code'] + ' - ' + obj['error']['info'] + '\n'));
- ISSUE:
line 119 character 76
: ['code'] is better written in dot notation. - Evidence:out.appendChild(document.createTextNode(' ! Api error: ' + obj['error']['code'] + ' - ' + obj['error']['info'] + '\n'));
- ISSUE:
line 119 character 98
: ['error'] is better written in dot notation. - Evidence:out.appendChild(document.createTextNode(' ! Api error: ' + obj['error']['code'] + ' - ' + obj['error']['info'] + '\n'));
- ISSUE:
line 119 character 107
: ['info'] is better written in dot notation. - Evidence:out.appendChild(document.createTextNode(' ! Api error: ' + obj['error']['code'] + ' - ' + obj['error']['info'] + '\n'));
- ISSUE:
line 122 character 10
: ['query'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 27
: ['query'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 36
: ['pageids'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 55
: ['query'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 64
: ['pages'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 77
: ['query'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 86
: ['pageids'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 109
: ['query'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 118
: ['pages'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 131
: ['query'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 140
: ['pageids'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 122 character 155
: ['deletetoken'] is better written in dot notation. - Evidence:if(!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]] || !obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken']) {
- ISSUE:
line 126 character 18
: ['query'] is better written in dot notation. - Evidence:var token = obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken'];
- ISSUE:
line 126 character 27
: ['pages'] is better written in dot notation. - Evidence:var token = obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken'];
- ISSUE:
line 126 character 40
: ['query'] is better written in dot notation. - Evidence:var token = obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken'];
- ISSUE:
line 126 character 49
: ['pageids'] is better written in dot notation. - Evidence:var token = obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken'];
- ISSUE:
line 126 character 64
: ['deletetoken'] is better written in dot notation. - Evidence:var token = obj['query']['pages'][obj['query']['pageids'][0]]['deletetoken'];
- ISSUE:
line 140 character 7
: eval can be harmful. - Evidence:eval("abdDeleteAftermath(" + req.responseText + ",'" + req.responseText.replace(/\'/g,"`") + "')");
- ISSUE:
line 142 character 4
: Missing semicolon. - Evidence:}
- ISSUE:
line 148 character 9
: ['error'] is better written in dot notation. - Evidence:if(obj['error']) {
- ISSUE:
line 149 character 69
: ['error'] is better written in dot notation. - Evidence:out.appendChild(document.createTextNode(' ! Api error: ' + obj['error']['code'] + ' - ' + obj['error']['info'] + '\n'));
- ISSUE:
line 149 character 78
: ['code'] is better written in dot notation. - Evidence:out.appendChild(document.createTextNode(' ! Api error: ' + obj['error']['code'] + ' - ' + obj['error']['info'] + '\n'));
- ISSUE:
line 149 character 100
: ['error'] is better written in dot notation. - Evidence:out.appendChild(document.createTextNode(' ! Api error: ' + obj['error']['code'] + ' - ' + obj['error']['info'] + '\n'));
- ISSUE:
line 149 character 109
: ['info'] is better written in dot notation. - Evidence:out.appendChild(document.createTextNode(' ! Api error: ' + obj['error']['code'] + ' - ' + obj['error']['info'] + '\n'));
- ISSUE:
line 150 character 43
: ['title'] is better written in dot notation. - Evidence:} else if(obj['delete'] && obj['delete']['title']) {
- ISSUE:
line 151 character 75
: ['title'] is better written in dot notation. - Evidence:out.appendChild(document.createTextNode(' > Page [[' + obj['delete']['title'] + ']] deleted\n'));
- ISSUE:
line 156 character 3
: Implied eval. Consider passing a function instead of a string. - Evidence:setTimeout('abdStart()',1000);
- ISSUE:
line 162 character 43
: Expected a conditional expression and instead saw an assignment. - Evidence:if (matches = re.exec(document.location)) {
- ISSUE:
line 1 character 11
: Missing semicolon. - Evidence:abdStart()
- ISSUE:
line 1 character 11
: Missing semicolon. - Evidence:abdStart()