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

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

jshint

1. ISSUE: line 27 character 24: Bad or unnecessary escaping. - Evidence: var usersignature = '\~\~\~\~';
2. ISSUE: line 27 character 26: Bad or unnecessary escaping. - Evidence: var usersignature = '\~\~\~\~';
3. ISSUE: line 27 character 28: Bad or unnecessary escaping. - Evidence: var usersignature = '\~\~\~\~';
4. ISSUE: line 27 character 30: Bad or unnecessary escaping. - Evidence: var usersignature = '\~\~\~\~';
5. ISSUE: line 61 character 20: Missing semicolon. - Evidence: var myButtons = {} // eigene Buttons?
6. ISSUE: line 369 character 55: Use '===' to compare with '0'. - Evidence: if ( mw.config.get( 'wgServer' ).indexOf('https://') == 0 ) {
7. ISSUE: line 378 character 32: The array literal notation [] is preferable. - Evidence: var UrlParameters = new Array ();
8. ISSUE: line 394 character 59: Use '===' to compare with '0'. - Evidence: var isArticle = mw.config.get('wgNamespaceNumber') == 0;
9. ISSUE: line 431 character 21: ['target'] is better written in dot notation. - Evidence: if (UrlParameters['target']) userPageOwner = UrlParameters['target'].split('+').join('_');
10. ISSUE: line 431 character 62: ['target'] is better written in dot notation. - Evidence: if (UrlParameters['target']) userPageOwner = UrlParameters['target'].split('+').join('_');
11. ISSUE: line 449 character 20: Use '!==' to compare with ''. - Evidence: if (userPageOwner != "")
12. ISSUE: line 450 character 77: Use '!==' to compare with 'null'. - Evidence: if (userPageOwner.match(/^(\d{1,3})\.(\d{1,3})\.(\d{1,3})\.(\d{1,3})$/) != null)
13. ISSUE: line 636 character 23: Use '===' to compare with ''. - Evidence: if (userPageOwner == "") return;
14. ISSUE: line 1099 character 2: document.write can be a form of eval. - Evidence: document.write(Quickbar);
15. ISSUE: line 1103 character 3: document.write can be a form of eval. - Evidence: document.write(
16. ISSUE: line 1126 character 2: document.write can be a form of eval. - Evidence: document.write('<div style="font-size:',qbfontsize,'%;">',xxx,'</div></div>');
17. ISSUE: line 1129 character 5: document.write can be a form of eval. - Evidence: document.write('<script> function _doopt (was) { switch (was) {', scrpt, '}}<\/script>');
18. ISSUE: line 1217 character 95: Missing semicolon. - Evidence: mw.toolbar.insertTags(untagEscape(tagOpen), untagEscape(tagClose), untagEscape(sampleText))
19. ISSUE: line 1222 character 26: Use '===' to compare with '0'. - Evidence: return (myNumber % 2 == 0);
20. ISSUE: line 1334 character 58: Use '===' to compare with '0'. - Evidence: if ( mw.config.get( 'wgServer' ).indexOf('https://') == 0 ) {
21. ISSUE: line 1337 character 25: 'import_url' is already defined. - Evidence: var import_url = '//' + lang + '.wikipedia.org' + mw.config.get( 'wgScriptPath' ) + '/index.php?title=' + unameprefix + seite + '&action=raw&ctype=text/javascript&smaxage=0' ;
22. ISSUE: line 1340 character 21: 'import_url' used out of scope. - Evidence: mw.loader.load( import_url );
23. ISSUE: line 1369 character 62: Use '===' to compare with '0'. - Evidence: if ( mw.config.get( 'wgServer' ).indexOf('https://') == 0 ) {
24. ISSUE: line 1372 character 29: 'import_url' is already defined. - Evidence: var import_url = '//' + lang + '.wikipedia.org' + mw.config.get( 'wgScriptPath' ) + '/index.php?title=User:' + page + '.js&action=raw&ctype=text/javascript&smaxage=0' ;
25. ISSUE: line 1375 character 25: 'import_url' used out of scope. - Evidence: mw.loader.load( import_url );