Commons:User scripts/users namespace/reports/User:Essjay~commonswiki/monobook.js
Jump to navigation
Jump to search
This is the report page for User:Essjay~commonswiki/monobook.js. It is bot-maintained. All manual changes might be overwritten.
jshint
[edit]- ISSUE:
line 71 character 15
: Implied eval. Consider passing a function instead of a string. - Evidence:timerID = setTimeout('showtime()', 100);
- ISSUE:
line 98 character 14
: Use '===' to compare with ''. - Evidence:if (target == '')
- ISSUE:
line 156 character 35
: Use '===' to compare with '0'. - Evidence:if(document.title.indexOf("User") == 0)
- ISSUE:
line 254 character 43
: Use '===' to compare with '0'. - Evidence:if (document.title.indexOf('Block user') == 0) // could stand to be more robust
- ISSUE:
line 266 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)
- ISSUE:
line 271 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
- ISSUE:
line 338 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
- ISSUE:
line 363 character 10
: Missing semicolon. - Evidence:}
- ISSUE:
line 367 character 10
: Missing semicolon. - Evidence:}
- ISSUE:
line 371 character 10
: Missing semicolon. - Evidence:}
- ISSUE:
line 376 character 114
: Missing semicolon. - Evidence:this.getText = function( id ) { return this[ id ].getElementsByTagName( 'a' )[ 0 ].firstChild.data }
- ISSUE:
line 377 character 120
: Missing semicolon. - Evidence:this.setText = function( id, txt ) { this[ id ].getElementsByTagName( 'a' )[ 0 ].firstChild.data = txt }
- ISSUE:
line 379 character 109
: Missing semicolon. - Evidence:this.getHref = function( id ) { return this[ id ].getElementsByTagName( 'a' )[ 0 ].href }
- ISSUE:
line 380 character 109
: Missing semicolon. - Evidence:this.setHref = function( id, url ) { this[ id ].getElementsByTagName( 'a' )[ 0 ].href = url }
- ISSUE:
line 385 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 )
- ISSUE:
line 386 character 50
: Missing semicolon. - Evidence:{ var node = document.getElementById( id )
- ISSUE:
line 463 character 3
: document.write can be a form of eval. - Evidence:document.writeln(lt+'script type="text/javascript" src="/w/index.php?title='+file+'&action=raw&ctype=text/javascript&dontcountme=s"'+gt+lt+'/script'+gt);
- ISSUE:
line 470 character 1
: document.write can be a form of eval. - Evidence:document.write('<script type="text/javascript" src="'
- ISSUE:
line 518 character 49
: Expected a conditional expression and instead saw an assignment. - Evidence:if (matches = re.exec(document.location)) {
- ISSUE:
line 525 character 6
: Unnecessary semicolon. - Evidence:};
- ISSUE:
line 1 character 11
: Missing semicolon. - Evidence:showtime()