Commons:User scripts/users namespace/reports/User:Patstuart/Extrascripts.js

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

jshint

1. ISSUE: line 14 character 13: Use '===' to compare with 'null'. - Evidence: if (pgName==null)
2. ISSUE: line 18 character 24: Use '===' to compare with '0'. - Evidence: if (textboxVal.length==0 || textboxVal.charAt(textboxVal.length-1)!='\n')
3. ISSUE: line 29 character 11: Use '===' to compare with 'null'. - Evidence: if (user==null)
4. ISSUE: line 48 character 15: Use '===' to compare with 'null'. - Evidence: if (uploader==null)
5. ISSUE: line 66 character 15: Use '===' to compare with 'null'. - Evidence: if (request == null)
6. ISSUE: line 75 character 16: Use '===' to compare with 'null'. - Evidence: if ( xml == null )
7. ISSUE: line 81 character 18: Use '===' to compare with 'null'. - Evidence: if (idArray==null)
8. ISSUE: line 88 character 14: Use '===' to compare with 'null'. - Evidence: if (id ==null)
9. ISSUE: line 95 character 13: Use '===' to compare with '0'. - Evidence: if (id==0) //no page
10. ISSUE: line 118 character 50: Missing semicolon. - Evidence: var el = document.getElementById('filehistory')
11. ISSUE: line 174 character 13: Use '===' to compare with 'null'. - Evidence: if (site==null)
12. ISSUE: line 176 character 71: Missing semicolon. - Evidence: } while(site.indexOf("Image:")==-1 && (alert("Illegal name"), true))
13. ISSUE: line 189 character 11: Use '===' to compare with 'null'. - Evidence: if (site==null)
14. ISSUE: line 192 character 11: Use '===' to compare with ''. - Evidence: if (site=="")
15. ISSUE: line 207 character 13: Use '===' to compare with 'null'. - Evidence: if (reason==null)
16. ISSUE: line 210 character 13: Use '===' to compare with ''. - Evidence: if (reason=="")
17. ISSUE: line 232 character 14: Use '===' to compare with 'null'. - Evidence: if (name == null) //cancel button
18. ISSUE: line 241 character 42: Use '===' to compare with '0'. - Evidence: if (str_trim(name).search(/I|image:/)==0)
19. ISSUE: line 243 character 15: Use '===' to compare with ''. - Evidence: } while(name=="" && (alert("Invalid image name"), true));
20. ISSUE: line 246 character 14: Use '===' to compare with 'null'. - Evidence: if (reason == null) //cancel button
21. ISSUE: line 250 character 13: Use '!==' to compare with ''. - Evidence: if (reason!="")
22. ISSUE: line 252 character 19: Missing semicolon. - Evidence: template+="}}\n"
23. ISSUE: line 255 character 9: Use '===' to compare with 'null'. - Evidence: if (ex==null)
24. ISSUE: line 257 character 9: Use '===' to compare with 'null'. - Evidence: if (ex==null)
25. ISSUE: line 260 character 10: Use '!==' to compare with 'null'. - Evidence: if (ex != null)
26. ISSUE: line 278 character 54: Missing semicolon. - Evidence: document.editform.wpTextbox1.value += '\n{{uncat}}'
27. ISSUE: line 286 character 94: Missing semicolon. - Evidence: document.editform.wpTextbox1.value = '{{subst:nld}}\n' + document.editform.wpTextbox1.value
28. ISSUE: line 293 character 94: Missing semicolon. - Evidence: document.editform.wpTextbox1.value = '{{subst:nsd}}\n' + document.editform.wpTextbox1.value
29. ISSUE: line 301 character 94: Missing semicolon. - Evidence: document.editform.wpTextbox1.value = '{{subst:npd}}\n' + document.editform.wpTextbox1.value
30. ISSUE: line 309 character 88: Missing semicolon. - Evidence: document.editform.wpTextbox1.value = '{{bsr}}\n' + document.editform.wpTextbox1.value
31. ISSUE: line 329 character 94: Missing semicolon. - Evidence: message = '[[Commons:Categories#People|Category sorting]]: "The" is not relevant to sort'
32. ISSUE: line 342 character 14: Use '===' to compare with 'null'. - Evidence: if(name==null) //cancel button
33. ISSUE: line 348 character 13: 'index' is already defined. - Evidence: var index = textboxVal.indexOf("[[Category:")
34. ISSUE: line 348 character 49: Missing semicolon. - Evidence: var index = textboxVal.indexOf("[[Category:")
35. ISSUE: line 365 character 42: Bad or unnecessary escaping. - Evidence: var ex = new RegExp("[0-9]{4} births((\s*)|(|.*))]]") .exec(textboxVal );
36. ISSUE: line 366 character 9: Use '===' to compare with 'null'. - Evidence: if (ex==null)
37. ISSUE: line 371 character 11: Use '!==' to compare with 'null'. - Evidence: if (ex!=null && ex.index==0)
38. ISSUE: line 371 character 29: Use '===' to compare with '0'. - Evidence: if (ex!=null && ex.index==0)
39. ISSUE: line 387 character 9: Use '===' to compare with 'null'. - Evidence: if (ex==null)
40. ISSUE: line 392 character 18: Use '===' to compare with 'null'. - Evidence: if (yeartxt==null) //cancel button
41. ISSUE: line 402 character 50: Use '===' to compare with '0'. - Evidence: } while(!(birth>=1800 && birth<2000 && (death==0 || (death>=1800 && death<=2008))) && (alert("Illegal Year"), true));
42. ISSUE: line 406 character 12: Use '!==' to compare with '0'. - Evidence: if (birth!=0)
43. ISSUE: line 423 character 14: Use '!==' to compare with '0'. - Evidence: if (death!=0)
44. ISSUE: line 440 character 74: Missing semicolon. - Evidence: yeartxt = "Actresses from the United States born in the 19th century"
45. ISSUE: line 442 character 73: Missing semicolon. - Evidence: yeartxt = "Actresses from the United States born in the " + year+"s"
46. ISSUE: line 462 character 27: Use '===' to compare with ''. - Evidence: if (wgCanonicalNamespace=="" || wgPageName == "User:Patstuart/test1")
47. ISSUE: line 465 character 16: Use '===' to compare with 'null'. - Evidence: if (catname==null) //cancel button pressed
48. ISSUE: line 480 character 54: Use '===' to compare with '0'. - Evidence: if (textboxVal.charAt(endcat)=='\n' && (nextcat==0 || textboxVal.charAt(nextcat-1)=='\n'))
49. ISSUE: line 484 character 20: Use '===' to compare with '0'. - Evidence: if (cats.length==0)
50. ISSUE: line 523 character 10: Creating global 'for' variable. Should be 'for (var i ...'. - Evidence: for (i in cats)
51. ISSUE: line 523 character 10: Too many errors. (69% scanned). - Evidence: undefined