Commons:User scripts/users namespace/reports/User:Animum/flickrreview.js

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

jshint

1. ISSUE: line 9 character 35: Bad or unnecessary escaping. - Evidence: var templates = new RegExp("(\{\{User:FlickreviewR.*\}\})|(\{\{[fF]lickrr?eview.*?\}\})", "g");
2. ISSUE: line 9 character 37: Bad or unnecessary escaping. - Evidence: var templates = new RegExp("(\{\{User:FlickreviewR.*\}\})|(\{\{[fF]lickrr?eview.*?\}\})", "g");
3. ISSUE: line 9 character 58: Bad or unnecessary escaping. - Evidence: var templates = new RegExp("(\{\{User:FlickreviewR.*\}\})|(\{\{[fF]lickrr?eview.*?\}\})", "g");
4. ISSUE: line 9 character 60: Bad or unnecessary escaping. - Evidence: var templates = new RegExp("(\{\{User:FlickreviewR.*\}\})|(\{\{[fF]lickrr?eview.*?\}\})", "g");
5. ISSUE: line 9 character 65: Bad or unnecessary escaping. - Evidence: var templates = new RegExp("(\{\{User:FlickreviewR.*\}\})|(\{\{[fF]lickrr?eview.*?\}\})", "g");
6. ISSUE: line 9 character 67: Bad or unnecessary escaping. - Evidence: var templates = new RegExp("(\{\{User:FlickreviewR.*\}\})|(\{\{[fF]lickrr?eview.*?\}\})", "g");
7. ISSUE: line 9 character 88: Bad or unnecessary escaping. - Evidence: var templates = new RegExp("(\{\{User:FlickreviewR.*\}\})|(\{\{[fF]lickrr?eview.*?\}\})", "g");
8. ISSUE: line 9 character 90: Bad or unnecessary escaping. - Evidence: var templates = new RegExp("(\{\{User:FlickreviewR.*\}\})|(\{\{[fF]lickrr?eview.*?\}\})", "g");
9. ISSUE: line 10 character 49: ['editform'] is better written in dot notation. - Evidence: var reviewed = templates.test(document.forms["editform"].wpTextbox1.value);
10. ISSUE: line 11 character 26: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
11. ISSUE: line 11 character 28: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
12. ISSUE: line 11 character 42: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
13. ISSUE: line 11 character 62: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
14. ISSUE: line 11 character 188: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
15. ISSUE: line 11 character 239: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
16. ISSUE: line 11 character 271: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
17. ISSUE: line 11 character 273: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
18. ISSUE: line 11 character 280: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
19. ISSUE: line 11 character 282: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
20. ISSUE: line 11 character 290: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
21. ISSUE: line 11 character 292: Bad or unnecessary escaping. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
22. ISSUE: line 11 character 175: Use '===' to compare with 'false'. - Evidence: var addedContent = "\{\{flickrreview\|" + wgUserName + "\|on " + date.getUTCFullYear() + "-" + zeroPad(date.getUTCMonth()+1) + "-" + zeroPad(date.getUTCDate()) + (isFree == false ? "\|" + otherLicense + (changedLicense.length > 0 ? "\|changed=" + changedLicense + "\}\}" : "\}\}") : "\}\}");
23. ISSUE: line 13 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpTextbox1.value = document.forms["editform"].wpTextbox1.value.replace(templates, addedContent);
24. ISSUE: line 13 character 69: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpTextbox1.value = document.forms["editform"].wpTextbox1.value.replace(templates, addedContent);
25. ISSUE: line 15 character 23: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpTextbox1.value += "\n" + addedContent;
26. ISSUE: line 17 character 19: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpSummary.value = "Image is" + (!isFree ? "n't" : "") + " compatible with the licensing policies of Wikimedia Commons.";
27. ISSUE: line 18 character 19: ['editform'] is better written in dot notation. - Evidence: document.forms["editform"].wpMinoredit.checked = true;
28. ISSUE: line 50 character 76: Script URL. - Evidence: mw.util.addPortletLink("p-cactions", "javascript:buildReviewForm()", "review", "ca-flickrreview");