Bugfix: moving time was not computed.

Introduced in 6be9d2477c (first released in v4.9.0)

Fixes #1756.

src/main/java/de/dennisguse/opentracks/data/models/TrackPoint.java

@@ -135,6 +135,10 @@ public class TrackPoint {
         return this;
     }
 
+    public boolean isIdleTriggered() {
+        return type == Type.IDLE;
+    }
+
     public boolean isSegmentManualStart() {
         return type == Type.SEGMENT_START_MANUAL;
     }

src/main/java/de/dennisguse/opentracks/stats/TrackStatistics.java

@@ -107,6 +107,7 @@ public class TrackStatistics {
      *
      * @param other another statistics data object
      */
+    //TODO Should be refactored to append only [mainly due to isIdle] (NOTE: This requires to use a custom value object for AggregatedStatistics; this is anyhow recommended).
     public void merge(TrackStatistics other) {
         if (startTime == null) {
             startTime = other.startTime;
@@ -119,6 +120,8 @@ public class TrackStatistics {
             stopTime = stopTime.isAfter(other.stopTime) ? stopTime : other.stopTime;
         }
 
+        isIdle = other.isIdle; //TODO This implicitly assumes append mode.
+
         if (avgHeartRate == null) {
             avgHeartRate = other.avgHeartRate;
         } else {

src/main/java/de/dennisguse/opentracks/stats/TrackStatisticsUpdater.java

@@ -26,6 +26,7 @@ import de.dennisguse.opentracks.data.models.HeartRate;
 import de.dennisguse.opentracks.data.models.Power;
 import de.dennisguse.opentracks.data.models.Speed;
 import de.dennisguse.opentracks.data.models.TrackPoint;
+import de.dennisguse.opentracks.settings.PreferencesUtils;
 
 /**
  * Updater for {@link TrackStatistics}.
@@ -56,11 +57,6 @@ public class TrackStatisticsUpdater {
         this(new TrackStatistics());
     }
 
-    /**
-     * Creates a new{@link TrackStatisticsUpdater} with a {@link TrackStatisticsUpdater} already existed.
-     *
-     * @param trackStatistics a {@link TrackStatisticsUpdater}
-     */
     public TrackStatisticsUpdater(TrackStatistics trackStatistics) {
         this.trackStatistics = trackStatistics;
         this.currentSegment = new TrackStatistics();
@@ -87,9 +83,6 @@ public class TrackStatisticsUpdater {
         trackPoints.stream().forEachOrdered(this::addTrackPoint);
     }
 
-    /**
-     *
-     */
     public void addTrackPoint(TrackPoint trackPoint) {
         if (trackPoint.isSegmentManualStart()) {
             reset(trackPoint);
@@ -151,18 +144,23 @@ public class TrackStatisticsUpdater {
                 movingDistance = trackPoint.distanceToPrevious(lastTrackPoint);
             }
             if (movingDistance != null) {
-                currentSegment.setIdle(false);
                 currentSegment.addTotalDistance(movingDistance);
             }
 
-            if (!currentSegment.isIdle() && !trackPoint.isSegmentManualStart()) {
-                if (lastTrackPoint != null) {
+            if (!currentSegment.isIdle()) {
+                if (!trackPoint.isSegmentManualStart() && lastTrackPoint != null) {
                     currentSegment.addMovingTime(trackPoint, lastTrackPoint);
                 }
             }
 
-            if (trackPoint.getType() == TrackPoint.Type.IDLE) {
+            if (trackPoint.isIdleTriggered()) {
                 currentSegment.setIdle(true);
+            } else if (currentSegment.isIdle()) {
+                // Shall we switch to non-idle?
+                if (movingDistance != null
+                        && movingDistance.greaterOrEqualThan(PreferencesUtils.getRecordingDistanceInterval())) {
+                    currentSegment.setIdle(false);
+                }
             }
 
             if (trackPoint.hasSpeed()) {