Skip to content

Commit fca7901

Browse files
Merge pull request #7165 from Countly/fix/memory-leak_for_master
Fix/memory leak for master
2 parents d410fef + fcabce8 commit fca7901

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

frontend/express/public/core/events/javascripts/countly.overview.models.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -305,10 +305,13 @@
305305
"method": "top_events",
306306
"period": "30days",
307307
"filter": filter,
308-
"limit": limit
308+
"limit": limit,
309+
"preventRequestAbort": true
309310
},
310311
dataType: "json",
311-
}, {"disableAutoCatch": true});//to be able to see if discarded
312+
}, {
313+
"disableAutoCatch": true,
314+
});//to be able to see if discarded
312315
},
313316
fetchMonitorEvents: function(context, period) {
314317
return CV.$.ajax({

frontend/express/public/core/events/javascripts/countly.overview.views.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -466,9 +466,6 @@
466466
self.calculateAllData();
467467
self.isLoading = false;
468468
}
469-
else {
470-
setTimeout(self.refresh(), 1000); //we have nothing retry.
471-
}
472469
}
473470
else {
474471
this.$root.$emit("cly-error", {message: errored});//show error
@@ -515,4 +512,4 @@
515512
component: EventsHomeWidget
516513
});
517514

518-
})();
515+
})();

0 commit comments

Comments
 (0)