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

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

jshint

1. ISSUE: line 4 character 45: Missing semicolon. - Evidence: document.getElementById('wpReason').focus()
2. ISSUE: line 24 character 3: document.write can be a form of eval. - Evidence: document.write('<script type="text/javascript" src="http://localhost/'
3. ISSUE: line 25 character 31: Missing semicolon. - Evidence: + extraJS + '"><\/script>')
4. ISSUE: line 43 character 28: Use '===' to compare with 'null'. - Evidence: if ( document.cookie == null || document.cookie.length == 0
5. ISSUE: line 43 character 62: Use '===' to compare with '0'. - Evidence: if ( document.cookie == null || document.cookie.length == 0
6. ISSUE: line 44 character 24: Use '===' to compare with 'null'. - Evidence: || cookie_name == null || cookie_name.length == 0)
7. ISSUE: line 44 character 54: Use '===' to compare with '0'. - Evidence: || cookie_name == null || cookie_name.length == 0)
8. ISSUE: line 59 character 14: Use '===' to compare with 'null'. - Evidence: if (form == null || form.elements == null || form.elements.length == 0) return;
9. ISSUE: line 59 character 39: Use '===' to compare with 'null'. - Evidence: if (form == null || form.elements == null || form.elements.length == 0) return;
10. ISSUE: line 59 character 71: Use '===' to compare with '0'. - Evidence: if (form == null || form.elements == null || form.elements.length == 0) return;
11. ISSUE: line 77 character 31: Expected a 'break' statement before 'case'. - Evidence: single_select = true;
12. ISSUE: line 82 character 33: Use '===' to compare with 'null'. - Evidence: if (element_value == null) {
13. ISSUE: line 96 character 29: Use '===' to compare with 'null'. - Evidence: if (element_value == null) element_value = "";
14. ISSUE: line 102 character 25: Use '!==' to compare with 'null'. - Evidence: if (element_value != null) {
15. ISSUE: line 104 character 23: Use '===' to compare with 'null'. - Evidence: if (this_item == null || this_item.length == 0) this_item = element.name;
16. ISSUE: line 104 character 51: Use '===' to compare with '0'. - Evidence: if (this_item == null || this_item.length == 0) this_item = element.name;
17. ISSUE: line 106 character 24: Use '===' to compare with 'null'. - Evidence: if (cookie_val == null || cookie_val.length == 0)
18. ISSUE: line 106 character 53: Use '===' to compare with '0'. - Evidence: if (cookie_val == null || cookie_val.length == 0)
19. ISSUE: line 112 character 20: Use '!==' to compare with 'null'. - Evidence: if (cookie_val != null) {
20. ISSUE: line 114 character 44: Use '!==' to compare with 'null'. - Evidence: + (cookie_attributes != null ? cookie_attributes : "");
21. ISSUE: line 143 character 21: Do not use Number as a constructor. - Evidence: value = new Number (val);
22. ISSUE: line 149 character 26: 'j' is already defined. - Evidence: for (var j = 0; j < numbers.length; j++)
23. ISSUE: line 150 character 28: Do not use Number as a constructor. - Evidence: value[j] = new Number (numbers[j]);
24. ISSUE: line 165 character 2: Missing semicolon. - Evidence: } // end FormRestorer