From 55ee2665a3aee98b552f5e0ae1f8cdcd8ab74eae Mon Sep 17 00:00:00 2001 From: Johannes Millan Date: Tue, 27 Feb 2018 19:51:42 +0100 Subject: [PATCH] fix: bug when huge amount of time was wrongly tracked when application is idle for a long time (e.g. when being on lock screen over several hours) --- app-src/scripts/main/global-services/time-tracking-s.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app-src/scripts/main/global-services/time-tracking-s.js b/app-src/scripts/main/global-services/time-tracking-s.js index 2e2e71532f..c28214454c 100644 --- a/app-src/scripts/main/global-services/time-tracking-s.js +++ b/app-src/scripts/main/global-services/time-tracking-s.js @@ -11,6 +11,7 @@ const IPC_EVENT_CURRENT_TASK_UPDATED = 'CHANGED_CURRENT_TASK'; const IPC_EVENT_IDLE_TIME = 'IDLE_TIME'; + const MAX_TRACKING_PERIOD_VAL = 60000; class TimeTracking { /* @ngInject */ @@ -45,7 +46,8 @@ let realPeriodDuration = moment.duration(now.diff(currentTrackingStart)) .asMilliseconds(); - if (!this.isIdle) { + // only track if not idle and interval is smaller than threshold + if (!this.isIdle && realPeriodDuration <= MAX_TRACKING_PERIOD_VAL) { this.Tasks.addTimeSpent(this.$rootScope.r.currentTask, realPeriodDuration); }