Commons:User scripts/users namespace/reports/User:MONUMENTA/MediaWiki:Gadget-ImageAnnotator.js

This is the report page for User:MONUMENTA/MediaWiki:Gadget-ImageAnnotator.js. It is bot-maintained. All manual changes might be overwritten.

jshint

1. ISSUE: line 435 character 39: Use '===' to compare with 'null'. - Evidence: if ( text == null ) { return ''; }
2. ISSUE: line 738 character 52: Use '!==' to compare with 'null'. - Evidence: if ( self.note.content != null ) {
3. ISSUE: line 1020 character 116: Don't make functions within a loop. - Evidence: !Array.exists( this.annotations, function ( note ) { return note.model.id == id; } )
4. ISSUE: line 1036 character 65: Use '===' to compare with '0'. - Evidence: if ( !this.main_div || this.main_div.length == 0 ) { this.main_div = null; } else { this.main_div = this.main_div[ 0 ]; }
5. ISSUE: line 1083 character 23: Don't make functions within a loop. - Evidence: } );
6. ISSUE: line 1231 character 29: 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 intersect_rectangles( a, b ) {
7. ISSUE: line 1261 character 71: Use '!==' to compare with 'null'. - Evidence: display !== 'none' && display != null &&
8. ISSUE: line 1403 character 55: Use '!==' to compare with 'null'. - Evidence: if ( !config || ImageAnnotator_config != null ) { return; }
9. ISSUE: line 1419 character 62: Use '!==' to compare with 'null'. - Evidence: self.haveAjax = ( LAPI.Ajax.getRequest() != null );
10. ISSUE: line 1867 character 29: 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 get_size( info ) {
11. ISSUE: line 1879 character 48: Use '!==' to compare with '0'. - Evidence: if ( i != 0 || !self.may_edit || !info.protection || mw.config.get( 'wgNamespaceNumber' ) != 6 ) { return; }
12. ISSUE: line 2005 character 40: 'item' is already defined. - Evidence: for ( var item in self.defaults ) {
13. ISSUE: line 2112 character 60: Use '!==' to compare with 'null'. - Evidence: IA.haveAjax = ( LAPI.Ajax.getRequest() != null );
14. ISSUE: line 2232 character 21: 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 track( evt ) {
15. ISSUE: line 2259 character 21: 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 pause( evt ) {
16. ISSUE: line 2265 character 21: 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 resume( evt ) {
17. ISSUE: line 2275 character 21: 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 stop_tracking( evt ) {
18. ISSUE: line 2322 character 21: 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 start_tracking( evt ) {
19. ISSUE: line 2355 character 21: 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 add_new( evt ) {
20. ISSUE: line 2621 character 31: Don't make functions within a loop. - Evidence: } );
21. ISSUE: line 3074 character 66: Use '!==' to compare with 'null'. - Evidence: self.haveAjax = ( LAPI.Ajax.getRequest() != null );