Commons:User scripts/users namespace/reports/User:MB-one/monobook.js
This is the report page for User:MB-one/monobook.js. It is bot-maintained. All manual changes might be overwritten.
jshint edit
- ISSUE:
line 71 character 12
: Use '===' to compare with ''. - Evidence:if ( t == "" || t == "(+)" ) continue ;
- ISSUE:
line 86 character 16
: Use '===' to compare with 'null'. - Evidence:if ( catline == null || typeof catline == 'undefined' ) return ;
- ISSUE:
line 102 character 2
: Missing semicolon. - Evidence:}
- ISSUE:
line 118 character 51
: Script URL. - Evidence:remove_link.href = "javascript:hotcat_remove(\"" + encodeURIComponent( cat_title ) + "\");" ;
- ISSUE:
line 129 character 51
: Script URL. - Evidence:modify_link.href = "javascript:hotcat_modify(\"" + mod_id + "\");" ;
- ISSUE:
line 166 character 27
: The array literal notation [] is preferable. - Evidence:var summary = new Array () ;
- ISSUE:
line 173 character 41
: Use '!==' to compare with 'null'. - Evidence:if ( typeof hrc != "undefined" && hrc != null && hrc != "" ) {
- ISSUE:
line 173 character 56
: Use '!==' to compare with ''. - Evidence:if ( typeof hrc != "undefined" && hrc != null && hrc != "" ) {
- ISSUE:
line 195 character 41
: Use '!==' to compare with 'null'. - Evidence:if ( typeof hnc != "undefined" && hnc != null && hnc != "" ) {
- ISSUE:
line 195 character 56
: Use '!==' to compare with ''. - Evidence:if ( typeof hnc != "undefined" && hnc != null && hnc != "" ) {
- ISSUE:
line 224 character 45
: Script URL. - Evidence:a_add.href = "javascript:hotcat_add_new()" ;
- ISSUE:
line 266 character 89
: ['HotCatDelay'] is better written in dot notation. - Evidence:text.onkeyup = function () { window.setTimeout("hotcat_text_changed();", JSconfig.keys['HotCatDelay'] ); } ;
- ISSUE:
line 266 character 38
: Implied eval. Consider passing a function instead of a string. - Evidence:text.onkeyup = function () { window.setTimeout("hotcat_text_changed();", JSconfig.keys['HotCatDelay'] ); } ;
- ISSUE:
line 296 character 10
: Use '===' to compare with ''. - Evidence:if ( v == "" ) {
- ISSUE:
line 366 character 10
: Use '!==' to compare with ''. - Evidence:if ( v != "" ) {
- ISSUE:
line 375 character 24
: Use '===' to compare with 'null'. - Evidence:if ( xml == null ) return ;
- ISSUE:
line 377 character 38
: The array literal notation [] is preferable. - Evidence:var titles = new Array () ;
- ISSUE:
line 388 character 28
: The array literal notation [] is preferable. - Evidence:var titles = new Array () ;
- ISSUE:
line 398 character 22
: Use '===' to compare with '0'. - Evidence:if ( titles.length == 0 ) {