Commons:User scripts/users namespace/reports/User:Philroc/common.js

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

jshint

1. ISSUE: line 1037 character 97: Don't make functions within a loop. - Evidence: && !Array.exists(this.annotations, function (note) { return note.model.id == id; })
2. ISSUE: line 1054 character 50: Use '===' to compare with '0'. - Evidence: if (!this.main_div || this.main_div.length == 0)
3. ISSUE: line 1104 character 10: Don't make functions within a loop. - Evidence: });
4. ISSUE: line 1260 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 1911 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 1923 character 25: Use '!==' to compare with '0'. - Evidence: if (i != 0 || !self.may_edit || !info.protection || mw.config.get('wgNamespaceNumber') != 6) return;
7. ISSUE: line 2050 character 23: 'item' is already defined. - Evidence: for (var item in self.defaults) {
8. ISSUE: line 2282 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 2309 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 2316 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 2327 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 2374 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 2407 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 2681 character 12: Don't make functions within a loop. - Evidence: });