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

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

jshint

1. ISSUE: line 74 character 1: Implied eval. Consider passing a function instead of a string. - Evidence: setTimeout ( "run_gallerypreview()" , 10 ) ;
2. ISSUE: line 87 character 34: The array literal notation [] is preferable. - Evidence: var classElements = new Array();
3. ISSUE: line 88 character 15: Use '===' to compare with 'null'. - Evidence: if ( node == null )
4. ISSUE: line 90 character 14: Use '===' to compare with 'null'. - Evidence: if ( tag == null )
5. ISSUE: line 109 character 15: Use '===' to compare with ''. - Evidence: if ( newstate == "" ) {
6. ISSUE: line 124 character 13: 'i' is already defined. - Evidence: for ( var i = 0 ; i < gp_image_xml.length ; i++ ) {
7. ISSUE: line 142 character 62: Use '===' to compare with 'true'. - Evidence: if (gp_image_xml[i].readyState==4 && gp_image_xml[i].running == true ) {
8. ISSUE: line 179 character 12: Use '===' to compare with ''. - Evidence: if ( image == "" ) {
9. ISSUE: line 187 character 12: Use '===' to compare with ''. - Evidence: if ( image == "" ) return ret ;
10. ISSUE: line 205 character 64: Use '===' to compare with 'true'. - Evidence: if (gp_xmlobjects[i].readyState==4 && gp_xmlobjects[i].running == true ) {
11. ISSUE: line 218 character 13: 'i' is already defined. - Evidence: for ( var i = 0 ; i < pages.length ; i++ ) {
12. ISSUE: line 221 character 10: Use '===' to compare with ''. - Evidence: if ( its == "" ) continue ; // No timestamp here
13. ISSUE: line 246 character 64: Use '===' to compare with 'true'. - Evidence: if (gp_xmlobjects[i].readyState==4 && gp_xmlobjects[i].running == true ) {
14. ISSUE: line 257 character 13: 'i' is already defined. - Evidence: for ( var i = 0 ; i < pages.length ; i++ ) {
15. ISSUE: line 261 character 20: Use '===' to compare with ''. - Evidence: if ( ( after_image == "" || image < after_image ) && image > pn) after_image = image ;
16. ISSUE: line 286 character 16: Use '===' to compare with ''. - Evidence: if ( timestamp == "" ) return ; // No timestamp, no point in looking further
17. ISSUE: line 294 character 7: 'i' is already defined. - Evidence: var i = gp_xmlobjects.length ;
18. ISSUE: line 317 character 7: 'i' is already defined. - Evidence: var i = gp_xmlobjects.length ;
19. ISSUE: line 337 character 9: Use '!==' to compare with 'null'. - Evidence: if ( ns != null && ns != "14" ) continue ; // Not a category. Strange.
20. ISSUE: line 342 character 15: Use '===' to compare with 'false'. - Evidence: if ( gp_found == false ) gp_status.innerHTML += "<div style='color:CD0000'>No non-trivial categories found. <a href='http://commons.wikimedia.org/wiki/Categories#Categorizing_your_uploads'>Please add some</a>.</div>" ;
21. ISSUE: line 353 character 9: Use '!==' to compare with 'null'. - Evidence: if ( ns != null && ns != "0" ) continue ; // Not gallery namespace
22. ISSUE: line 357 character 16: Use '===' to compare with 'false'. - Evidence: if ( gp_found2 == false ) gp_status.innerHTML += "<div style='color:#CD0000'>galerie nenalezeny</div>";
23. ISSUE: line 387 character 25: The array literal notation [] is preferable. - Evidence: gp_image_xml = new Array() ;
24. ISSUE: line 389 character 26: The array literal notation [] is preferable. - Evidence: gp_xmlobjects = new Array() ;
25. ISSUE: line 1 character 21: Missing semicolon. - Evidence: run_gallerypreview()