Commons:User scripts/users namespace/reports/User:Brian/monobook.js

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

jshint

1. ISSUE: line 31 character 14: Use '===' to compare with ''. - Evidence: if (target == '')
2. ISSUE: line 100 character 31: Use '===' to compare with '0'. - Evidence: if (title.indexOf('Image:') == 0)
3. ISSUE: line 102 character 35: Use '===' to compare with '0'. - Evidence: else if (title.indexOf('User:') == 0
4. ISSUE: line 103 character 38: Use '===' to compare with '0'. - Evidence: || title.indexOf('User_talk:') == 0)
5. ISSUE: line 105 character 40: Use '===' to compare with '0'. - Evidence: else if (title.indexOf('Block_user') == 0) // could stand to be more robust
6. ISSUE: line 146 character 16: eval can be harmful. - Evidence: var info = eval('(' + req.responseText + ')');
7. ISSUE: line 147 character 25: ['query'] is better written in dot notation. - Evidence: for (var key in info['query']['pages'])
8. ISSUE: line 147 character 34: ['pages'] is better written in dot notation. - Evidence: for (var key in info['query']['pages'])
9. ISSUE: line 149 character 24: ['query'] is better written in dot notation. - Evidence: var page = info['query']['pages'][key];
10. ISSUE: line 149 character 33: ['pages'] is better written in dot notation. - Evidence: var page = info['query']['pages'][key];
11. ISSUE: line 150 character 25: ['edittoken'] is better written in dot notation. - Evidence: var token = page['edittoken'];
12. ISSUE: line 151 character 27: ['revisions'] is better written in dot notation. - Evidence: var content = page['revisions'][0]['*'];
13. ISSUE: line 152 character 28: ['revisions'] is better written in dot notation. - Evidence: var editTime = page['revisions'][0]['timestamp'].replace(/[^0-9]/g, '');
14. ISSUE: line 152 character 44: ['timestamp'] is better written in dot notation. - Evidence: var editTime = page['revisions'][0]['timestamp'].replace(/[^0-9]/g, '');
15. ISSUE: line 189 character 9: document.write can be a form of eval. - Evidence: document.write(req.responseText);
16. ISSUE: line 205 character 57: Script URL. - Evidence: a.setAttribute('href', 'javascript:void(OTRS())');