Commons:User scripts/users namespace/reports/User:GregAsche~commonswiki/monobook.js

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

jshint

1. ISSUE: line 65 character 14: Use '===' to compare with ''. - Evidence: if (target == '')
2. ISSUE: line 121 character 54: Script URL. - Evidence: tabs.appendChild(addlilink('javascript:arin_look("' + editlk + '")', 'Arin'));
3. ISSUE: line 127 character 39: Use '===' to compare with '0'. - Evidence: if (document.title.indexOf('User:') == 0
4. ISSUE: line 128 character 47: Use '===' to compare with '0'. - Evidence: || document.title.indexOf('User talk:') == 0)
5. ISSUE: line 130 character 49: Use '===' to compare with '0'. - Evidence: else if (document.title.indexOf('Block user') == 0) // could stand to be more robust
6. ISSUE: line 144 character 5: 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 SoFixItInit() // pre-load, (don't want to slow down loading of article's content, though)
7. ISSUE: line 149 character 5: 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 SoFixItLoad() // post-load
8. ISSUE: line 198 character 44: Script URL. - Evidence: 'javascript:void InlineKate( "' + userName + '" )' );
9. ISSUE: line 216 character 5: 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 PortletMenu( id ) // constructor
10. ISSUE: line 241 character 10: Missing semicolon. - Evidence: }
11. ISSUE: line 245 character 10: Missing semicolon. - Evidence: }
12. ISSUE: line 249 character 10: Missing semicolon. - Evidence: }
13. ISSUE: line 254 character 114: Missing semicolon. - Evidence: this.getText = function( id ) { return this[ id ].getElementsByTagName( 'a' )[ 0 ].firstChild.data }
14. ISSUE: line 255 character 120: Missing semicolon. - Evidence: this.setText = function( id, txt ) { this[ id ].getElementsByTagName( 'a' )[ 0 ].firstChild.data = txt }
15. ISSUE: line 257 character 109: Missing semicolon. - Evidence: this.getHref = function( id ) { return this[ id ].getElementsByTagName( 'a' )[ 0 ].href }
16. ISSUE: line 258 character 109: Missing semicolon. - Evidence: this.setHref = function( id, url ) { this[ id ].getElementsByTagName( 'a' )[ 0 ].href = url }
17. ISSUE: line 263 character 5: 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 InlineKate( user )
18. ISSUE: line 295 character 78: Script URL. - Evidence: a.href = "javascript:void RemoveNode('inlineKateDiv')";
19. ISSUE: line 317 character 5: 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 RemoveNode( id )
20. ISSUE: line 318 character 50: Missing semicolon. - Evidence: { var node = document.getElementById( id )
21. ISSUE: line 322 character 5: 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 UTCTime()