Commons:User scripts/users namespace/reports/User:Dr Brains/NewPagePatrol.js

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

jshint

1. ISSUE: line 36 character 21: Use '===' to compare with 'null'. - Evidence: if (npp_enabled == null) {
2. ISSUE: line 39 character 23: Use '===' to compare with 'null'. - Evidence: if (npp_num_pages == null) {
3. ISSUE: line 42 character 21: Use '===' to compare with 'null'. - Evidence: if (npp_refresh == null) {
4. ISSUE: line 45 character 26: Use '===' to compare with 'null'. - Evidence: if (npp_num_idle_req == null) {
5. ISSUE: line 95 character 27: Missing semicolon. - Evidence: Link.id = 'l-newpages'
6. ISSUE: line 97 character 55: Script URL. - Evidence: Link.href = "javascript:NewPagePatrol_closeBox();";
7. ISSUE: line 117 character 58: Script URL. - Evidence: Link.href = "javascript:NewPagePatrol_openBox();";
8. ISSUE: line 132 character 59: Script URL. - Evidence: Link.href = "javascript:NewPagePatrol_closeBox();";
9. ISSUE: line 139 character 19: Use '===' to compare with 'false'. - Evidence: if(npp_enabled==false) return ;
10. ISSUE: line 153 character 22: Use '===' to compare with 'null'. - Evidence: if(ObjectXML == null) return ;
11. ISSUE: line 154 character 23: Use '===' to compare with 'false'. - Evidence: if(npp_enabled==false) return ;
12. ISSUE: line 165 character 64: Use '!==' to compare with 'null'. - Evidence: var patrolled = items[i].getAttribute('patrolled') != null;
13. ISSUE: line 185 character 31: Implied eval. Consider passing a function instead of a string. - Evidence: npp_NextRequest = setTimeout("NewPagePatrol_Update()", npp_refresh * 1000);
14. ISSUE: line 187 character 6: Missing semicolon. - Evidence: }
15. ISSUE: line 1 character 23: Missing semicolon. - Evidence: NewPagePatrol_Update()