Commons:User scripts/users namespace/reports/User:Mr.Z-man/closeDR.js
Jump to navigation
Jump to search
This is the report page for User:Mr.Z-man/closeDR.js. It is bot-maintained. All manual changes might be overwritten.
jshint[edit]
- ISSUE:
line 7 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function closeDR_init() {
- ISSUE:
line 26 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function closeDR_other() {
- ISSUE:
line 34 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function closeDR_otherdelete() {
- ISSUE:
line 36 character 3
: Read only. - Evidence:close = document.getElementById('closeDR_otherclose').value;
- ISSUE:
line 39 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function closeDR_otherkeep() {
- ISSUE:
line 40 character 3
: Read only. - Evidence:close = document.getElementById('closeDR_otherclose').value;
- ISSUE:
line 43 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function closeDR_close(close) {
- ISSUE:
line 50 character 14
: Use '!==' to compare with ''. - Evidence:if (reason != '') {
- ISSUE:
line 53 character 57
: Use '===' to compare with 'true'. - Evidence:if (document.getElementById('closeDR_speedy').checked == true) {
- ISSUE:
line 70 character 23
: Missing semicolon. - Evidence:var articlepage = ''
- ISSUE:
line 81 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function closeDR_getPageText(title) {
- ISSUE:
line 86 character 18
: eval can be harmful. - Evidence:var response = eval('(' + req.responseText + ')');
- ISSUE:
line 87 character 20
: ['query'] is better written in dot notation. - Evidence:pageid = response['query']['pageids'][0];
- ISSUE:
line 87 character 29
: ['pageids'] is better written in dot notation. - Evidence:pageid = response['query']['pageids'][0];
- ISSUE:
line 90 character 15
: Variables should not be deleted. - Evidence:delete req;
- ISSUE:
line 93 character 22
: ['query'] is better written in dot notation. - Evidence:pagetext = response['query']['pages'][pageid]['revisions'][0]['*'];
- ISSUE:
line 93 character 31
: ['pages'] is better written in dot notation. - Evidence:pagetext = response['query']['pages'][pageid]['revisions'][0]['*'];
- ISSUE:
line 93 character 48
: ['revisions'] is better written in dot notation. - Evidence:pagetext = response['query']['pages'][pageid]['revisions'][0]['*'];
- ISSUE:
line 94 character 13
: Variables should not be deleted. - Evidence:delete req;
- ISSUE:
line 98 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function closeDR_getToken(show) {
- ISSUE:
line 105 character 18
: eval can be harmful. - Evidence:var response = eval('(' + req.responseText + ')');
- ISSUE:
line 106 character 20
: ['query'] is better written in dot notation. - Evidence:pageid = response['query']['pageids'][0];
- ISSUE:
line 106 character 29
: ['pageids'] is better written in dot notation. - Evidence:pageid = response['query']['pageids'][0];
- ISSUE:
line 107 character 19
: ['query'] is better written in dot notation. - Evidence:token = response['query']['pages'][pageid]['edittoken'];
- ISSUE:
line 107 character 28
: ['pages'] is better written in dot notation. - Evidence:token = response['query']['pages'][pageid]['edittoken'];
- ISSUE:
line 107 character 45
: ['edittoken'] is better written in dot notation. - Evidence:token = response['query']['pages'][pageid]['edittoken'];
- ISSUE:
line 108 character 13
: Variables should not be deleted. - Evidence:delete req;
- ISSUE:
line 114 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function closeDR_editPage(title, newtext, token, summary) {
- ISSUE:
line 128 character 18
: eval can be harmful. - Evidence:response = eval('(' + req.responseText + ')');
- ISSUE:
line 130 character 21
: ['edit'] is better written in dot notation. - Evidence:if (response['edit']['result'] == "Success") {
- ISSUE:
line 130 character 29
: ['result'] is better written in dot notation. - Evidence:if (response['edit']['result'] == "Success") {
- ISSUE:
line 133 character 221
: ['error'] is better written in dot notation. - Evidence:document.getElementById('closeDR_edit'+escape(title)).innerHTML = '<div style="color:red"><b>Edit failed on <a href="/wiki/'+encodeURI(title)+'" title="'+title+'">'+title+'</a></b></div>. Error info:' +response['error']['code'] + ' : ' + response['error']['info'];
- ISSUE:
line 133 character 230
: ['code'] is better written in dot notation. - Evidence:document.getElementById('closeDR_edit'+escape(title)).innerHTML = '<div style="color:red"><b>Edit failed on <a href="/wiki/'+encodeURI(title)+'" title="'+title+'">'+title+'</a></b></div>. Error info:' +response['error']['code'] + ' : ' + response['error']['info'];
- ISSUE:
line 133 character 257
: ['error'] is better written in dot notation. - Evidence:document.getElementById('closeDR_edit'+escape(title)).innerHTML = '<div style="color:red"><b>Edit failed on <a href="/wiki/'+encodeURI(title)+'" title="'+title+'">'+title+'</a></b></div>. Error info:' +response['error']['code'] + ' : ' + response['error']['info'];
- ISSUE:
line 133 character 266
: ['info'] is better written in dot notation. - Evidence:document.getElementById('closeDR_edit'+escape(title)).innerHTML = '<div style="color:red"><b>Edit failed on <a href="/wiki/'+encodeURI(title)+'" title="'+title+'">'+title+'</a></b></div>. Error info:' +response['error']['code'] + ' : ' + response['error']['info'];
- ISSUE:
line 140 character 17
: Variables should not be deleted. - Evidence:delete req;
- ISSUE:
line 142 character 4
: Missing semicolon. - Evidence:}
- ISSUE:
line 143 character 19
: Missing semicolon. - Evidence:req.send(params)
- ISSUE:
line 145 character 1
: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence:function closeDR_deletePage(title, token, summary, isredirect, retry) {
- ISSUE:
line 161 character 18
: eval can be harmful. - Evidence:response = eval('(' + req.responseText + ')');
- ISSUE:
line 163 character 31
: ['title'] is better written in dot notation. - Evidence:if (response['delete']['title']) {
- ISSUE:
line 168 character 21
: ['error'] is better written in dot notation. - Evidence:if (response['error']['code'] == "missingtitle") {
- ISSUE:
line 168 character 30
: ['code'] is better written in dot notation. - Evidence:if (response['error']['code'] == "missingtitle") {
- ISSUE:
line 172 character 219
: ['error'] is better written in dot notation. - Evidence:document.getElementById('closeDR_delete'+escape(title)).innerHTML = '<span style="color:red"><b>Deletion error on <a href="/wiki/'+encodeURI(title)+'" title="'+title+'">'+title+'</a>. Error info:' +response['error']['code'] + ' : ' + response['error']['info'] + '</b></span>';
- ISSUE:
line 172 character 228
: ['code'] is better written in dot notation. - Evidence:document.getElementById('closeDR_delete'+escape(title)).innerHTML = '<span style="color:red"><b>Deletion error on <a href="/wiki/'+encodeURI(title)+'" title="'+title+'">'+title+'</a>. Error info:' +response['error']['code'] + ' : ' + response['error']['info'] + '</b></span>';
- ISSUE:
line 172 character 255
: ['error'] is better written in dot notation. - Evidence:document.getElementById('closeDR_delete'+escape(title)).innerHTML = '<span style="color:red"><b>Deletion error on <a href="/wiki/'+encodeURI(title)+'" title="'+title+'">'+title+'</a>. Error info:' +response['error']['code'] + ' : ' + response['error']['info'] + '</b></span>';
- ISSUE:
line 172 character 264
: ['info'] is better written in dot notation. - Evidence:document.getElementById('closeDR_delete'+escape(title)).innerHTML = '<span style="color:red"><b>Deletion error on <a href="/wiki/'+encodeURI(title)+'" title="'+title+'">'+title+'</a>. Error info:' +response['error']['code'] + ' : ' + response['error']['info'] + '</b></span>';
- ISSUE:
line 177 character 17
: Variables should not be deleted. - Evidence:delete req;
- ISSUE:
line 179 character 4
: Missing semicolon. - Evidence:}
- ISSUE:
line 180 character 19
: Missing semicolon. - Evidence:req.send(params)
- ISSUE:
line 180 character 19
: Too many errors. (95% scanned). - Evidence:undefined