Merge pull request #8 from SonamChugh13/omer

Fixing nested if issue in TrackStatisticsUpdater
SonamChugh13 · Feb 11, 2024 · 6b7becf · 6b7becf
2 parents e1e773f + 0285876
commit 6b7becf
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/main/java/de/dennisguse/opentracks/stats/TrackStatisticsUpdater.java b/src/main/java/de/dennisguse/opentracks/stats/TrackStatisticsUpdater.java
@@ -141,11 +141,8 @@ public void addTrackPoint(TrackPoint trackPoint) {
                 currentSegment.addTotalDistance(movingDistance);
             }
 
-            if (!currentSegment.isIdle() && !trackPoint.isSegmentManualStart()) {
-                if (lastTrackPoint != null) {
-                    currentSegment.addMovingTime(trackPoint, lastTrackPoint);
-                }
-            }
+            if (!currentSegment.isIdle() && !trackPoint.isSegmentManualStart() && lastTrackPoint != null) {
+                currentSegment.addMovingTime(trackPoint, lastTrackPoint);
 
             if (trackPoint.getType() == TrackPoint.Type.IDLE) {
                 currentSegment.setIdle(true);