Commons:User scripts/users namespace/reports/User:TheDJ/Gadget-ImageAnnotator.js

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

jshint

1. ISSUE: line 1053 character 97: Don't make functions within a loop. - Evidence: && !Array.exists(this.annotations, function (note) { return note.model.id == id; })
2. ISSUE: line 1070 character 50: Use '===' to compare with '0'. - Evidence: if (!this.main_div || this.main_div.length == 0)
3. ISSUE: line 1120 character 10: Don't make functions within a loop. - Evidence: });
4. ISSUE: line 1283 character 11: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function intersect_rectangles (a, b) {
5. ISSUE: line 1931 character 11: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function get_size (info) {
6. ISSUE: line 1943 character 25: Use '!==' to compare with '0'. - Evidence: if (i != 0 || !self.may_edit || !info.protection || wgNamespaceNumber != 6) return;
7. ISSUE: line 2073 character 23: 'item' is already defined. - Evidence: for (var item in self.defaults) {
8. ISSUE: line 2310 character 5: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function track (evt) {
9. ISSUE: line 2337 character 5: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function pause (evt) {
10. ISSUE: line 2344 character 5: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function resume (evt) {
11. ISSUE: line 2355 character 5: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function stop_tracking (evt) {
12. ISSUE: line 2402 character 5: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function start_tracking (evt)
13. ISSUE: line 2436 character 5: Function declarations should not be placed in blocks. Use a function expression or move the statement to the top of the outer function. - Evidence: function add_new (evt) {
14. ISSUE: line 2646 character 38: Use '===' to compare with '0'. - Evidence: if (!notes || notes.length == 0) continue;
15. ISSUE: line 2651 character 42: Use '===' to compare with '0'. - Evidence: if (!viewers || viewers.length == 0) continue;
16. ISSUE: line 2706 character 12: Don't make functions within a loop. - Evidence: });