karol/OpenTracks
1
0
Fork 0
forked from upstream-mirrors/OpenTracks
Code Pull Requests Actions Releases 2 Activity

Bugfix: TrackDataHub needs to use the same TrackStatisticsUpdater for a whole track (incl. newly received TrackPoints).

Browse Source 
Fixes #745.

Introduced in https://github.com/OpenTracksApp/OpenTracks/commit/887d4700d4f8fbf10563834e7498388a3213007e
This commit is contained in:
Dennis Guse
2021-05-08 14:28:24 +02:00
parent e822172f58
commit 48b6438c06
1 changed files with 3 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/main/java/de/dennisguse/opentracks/content/TrackDataHub.java
+3 -8
View File
@@ -91,6 +91,7 @@ public class TrackDataHub {
private int numLoadedPoints; private int numLoadedPoints;
private TrackPoint.Id firstSeenTrackPointId; private TrackPoint.Id firstSeenTrackPointId;
private TrackPoint.Id lastSeenTrackPointId; private TrackPoint.Id lastSeenTrackPointId;
private TrackStatisticsUpdater trackStatisticsUpdater;
// Registered listeners // Registered listeners
private ContentObserver tracksTableObserver; private ContentObserver tracksTableObserver;
@@ -145,12 +146,6 @@ public class TrackDataHub {
} }
}; };
contentResolver.registerContentObserver(TrackPointsColumns.CONTENT_URI_BY_ID, false, trackPointsTableObserver); contentResolver.registerContentObserver(TrackPointsColumns.CONTENT_URI_BY_ID, false, trackPointsTableObserver);
handler.post(() -> {
if (started) {
loadDataForAll();
}
});
} }
public void stop() { public void stop() {
@@ -160,6 +155,7 @@ public class TrackDataHub {
} }
started = false; started = false;
trackStatisticsUpdater = null;
//Unregister listeners //Unregister listeners
ContentResolver contentResolver = context.getContentResolver(); ContentResolver contentResolver = context.getContentResolver();
@@ -355,8 +351,6 @@ public class TrackDataHub {
next = new TrackPoint.Id(localLastSeenTrackPointIdId.getId() + 1); //TODO startTrackPointId + 1 is an assumption assumption; should be derived from the DB. next = new TrackPoint.Id(localLastSeenTrackPointIdId.getId() + 1); //TODO startTrackPointId + 1 is an assumption assumption; should be derived from the DB.
} }
TrackStatisticsUpdater trackStatisticsUpdater = new TrackStatisticsUpdater();
TrackPoint trackPoint = null; TrackPoint trackPoint = null;
try (TrackPointIterator trackPointIterator = contentProviderUtils.getTrackPointLocationIterator(selectedTrackId, next)) { try (TrackPointIterator trackPointIterator = contentProviderUtils.getTrackPointLocationIterator(selectedTrackId, next)) {
@@ -439,6 +433,7 @@ public class TrackDataHub {
numLoadedPoints = 0; numLoadedPoints = 0;
firstSeenTrackPointId = null; firstSeenTrackPointId = null;
lastSeenTrackPointId = null; lastSeenTrackPointId = null;
trackStatisticsUpdater = new TrackStatisticsUpdater();
} }
public void setRecordingStatus(TrackRecordingService.RecordingStatus recordingStatus) { public void setRecordingStatus(TrackRecordingService.RecordingStatus recordingStatus) {