Commons:User scripts/users namespace/reports/User:Fetchcomms/speedy.js

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

jshint

1. ISSUE: line 14 character 16: Bad or unnecessary escaping. - Evidence: 'open': '{\{speedy|',
2. ISSUE: line 68 character 69: Bad or unnecessary escaping. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:insertTagOnPage("{\{example|","}}","example");', 'example tag');
3. ISSUE: line 68 character 97: Script URL. - Evidence: mw.util.addPortletLink('p-cactions', 'javascript:insertTagOnPage("{\{example|","}}","example");', 'example tag');
4. ISSUE: line 73 character 40: Use '!==' to compare with ''. - Evidence: if (inb.close && inb.close != '') {
5. ISSUE: line 74 character 76: Script URL. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '","' + inb['close'] + '","' + inb['def'] + '");', inb['label']);
6. ISSUE: line 74 character 82: ['usernote'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '","' + inb['close'] + '","' + inb['def'] + '");', inb['label']);
7. ISSUE: line 74 character 108: ['img_summary'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '","' + inb['close'] + '","' + inb['def'] + '");', inb['label']);
8. ISSUE: line 74 character 137: ['talk_summary'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '","' + inb['close'] + '","' + inb['def'] + '");', inb['label']);
9. ISSUE: line 74 character 167: ['open'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '","' + inb['close'] + '","' + inb['def'] + '");', inb['label']);
10. ISSUE: line 74 character 189: ['close'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '","' + inb['close'] + '","' + inb['def'] + '");', inb['label']);
11. ISSUE: line 74 character 212: ['def'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '","' + inb['close'] + '","' + inb['def'] + '");', inb['label']);
12. ISSUE: line 74 character 232: ['label'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '","' + inb['close'] + '","' + inb['def'] + '");', inb['label']);
13. ISSUE: line 76 character 76: Script URL. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '");', inb['label']);
14. ISSUE: line 76 character 82: ['usernote'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '");', inb['label']);
15. ISSUE: line 76 character 108: ['img_summary'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '");', inb['label']);
16. ISSUE: line 76 character 137: ['talk_summary'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '");', inb['label']);
17. ISSUE: line 76 character 167: ['open'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '");', inb['label']);
18. ISSUE: line 76 character 188: ['label'] is better written in dot notation. - Evidence: mw.util.addPortletLink(toollocation, 'javascript:insertTagOnPage("' + inb['usernote'] + '","' + inb['img_summary'] + '","' + inb['talk_summary'] + '","' + inb['open'] + '");', inb['label']);
19. ISSUE: line 85 character 20: Use '===' to compare with 'null'. - Evidence: if (tagtxt == null) return
20. ISSUE: line 85 character 35: Missing semicolon. - Evidence: if (tagtxt == null) return
21. ISSUE: line 88 character 17: 'tag' is already defined. - Evidence: var tag = tagstart;
22. ISSUE: line 111 character 13: eval can be harmful. - Evidence: eval("insertTagOnPageEdit(" + req.responseText + ",'" + req.responseText.replace(/\'/g, "`") + "','" + tag + "','" + usernote + "','" + img_summary +"','" + talk_summary + "')");
23. ISSUE: line 113 character 6: Missing semicolon. - Evidence: }
24. ISSUE: line 118 character 12: ['error'] is better written in dot notation. - Evidence: if (obj['error']) {
25. ISSUE: line 119 character 30: ['error'] is better written in dot notation. - Evidence: alert('Error: ' + obj['error']['code'] + ' - ' + obj['error']['info']);
26. ISSUE: line 119 character 39: ['code'] is better written in dot notation. - Evidence: alert('Error: ' + obj['error']['code'] + ' - ' + obj['error']['info']);
27. ISSUE: line 119 character 61: ['error'] is better written in dot notation. - Evidence: alert('Error: ' + obj['error']['code'] + ' - ' + obj['error']['info']);
28. ISSUE: line 119 character 70: ['info'] is better written in dot notation. - Evidence: alert('Error: ' + obj['error']['code'] + ' - ' + obj['error']['info']);
29. ISSUE: line 122 character 13: ['query'] is better written in dot notation. - Evidence: if (!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]]) {
30. ISSUE: line 122 character 30: ['query'] is better written in dot notation. - Evidence: if (!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]]) {
31. ISSUE: line 122 character 39: ['pageids'] is better written in dot notation. - Evidence: if (!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]]) {
32. ISSUE: line 122 character 58: ['query'] is better written in dot notation. - Evidence: if (!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]]) {
33. ISSUE: line 122 character 67: ['pages'] is better written in dot notation. - Evidence: if (!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]]) {
34. ISSUE: line 122 character 80: ['query'] is better written in dot notation. - Evidence: if (!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]]) {
35. ISSUE: line 122 character 89: ['pageids'] is better written in dot notation. - Evidence: if (!obj['query'] || !obj['query']['pageids'] || !obj['query']['pages'][obj['query']['pageids'][0]]) {
36. ISSUE: line 126 character 19: ['query'] is better written in dot notation. - Evidence: var page = obj['query']['pages'][obj['query']['pageids'][0]];
37. ISSUE: line 126 character 28: ['pages'] is better written in dot notation. - Evidence: var page = obj['query']['pages'][obj['query']['pageids'][0]];
38. ISSUE: line 126 character 41: ['query'] is better written in dot notation. - Evidence: var page = obj['query']['pages'][obj['query']['pageids'][0]];
39. ISSUE: line 126 character 50: ['pageids'] is better written in dot notation. - Evidence: var page = obj['query']['pages'][obj['query']['pageids'][0]];
40. ISSUE: line 127 character 14: ['edittoken'] is better written in dot notation. - Evidence: if (!page['edittoken'] || !page['revisions'] || !page['revisions'][0] || !page['revisions'][0]['timestamp']) {
41. ISSUE: line 127 character 36: ['revisions'] is better written in dot notation. - Evidence: if (!page['edittoken'] || !page['revisions'] || !page['revisions'][0] || !page['revisions'][0]['timestamp']) {
42. ISSUE: line 127 character 58: ['revisions'] is better written in dot notation. - Evidence: if (!page['edittoken'] || !page['revisions'] || !page['revisions'][0] || !page['revisions'][0]['timestamp']) {
43. ISSUE: line 127 character 83: ['revisions'] is better written in dot notation. - Evidence: if (!page['edittoken'] || !page['revisions'] || !page['revisions'][0] || !page['revisions'][0]['timestamp']) {
44. ISSUE: line 127 character 99: ['timestamp'] is better written in dot notation. - Evidence: if (!page['edittoken'] || !page['revisions'] || !page['revisions'][0] || !page['revisions'][0]['timestamp']) {
45. ISSUE: line 131 character 25: ['edittoken'] is better written in dot notation. - Evidence: var edittoken = page['edittoken'];
46. ISSUE: line 132 character 25: ['revisions'] is better written in dot notation. - Evidence: var timestamp = page['revisions'][0]['timestamp'];
47. ISSUE: line 132 character 41: ['timestamp'] is better written in dot notation. - Evidence: var timestamp = page['revisions'][0]['timestamp'];
48. ISSUE: line 133 character 26: ['starttimestamp'] is better written in dot notation. - Evidence: var startstamp = page['starttimestamp'];
49. ISSUE: line 136 character 84: Missing semicolon. - Evidence: if (startstamp) params += '&starttimestamp=' + startstamp.replace(/[^\d]/g, '')
50. ISSUE: line 147 character 17: eval can be harmful. - Evidence: eval("insertTagOnPageEditHandler(" + req.responseText + ",'" + req.responseText.replace(/\'/g, "`") + "')");
51. ISSUE: line 147 character 17: Too many errors. (72% scanned). - Evidence: undefined