diff --git a/src/main/java/de/dennisguse/opentracks/services/LocationManagerConnector.java b/src/main/java/de/dennisguse/opentracks/services/LocationManagerConnector.java index 6c6885ff7..8914a9b9e 100644 --- a/src/main/java/de/dennisguse/opentracks/services/LocationManagerConnector.java +++ b/src/main/java/de/dennisguse/opentracks/services/LocationManagerConnector.java @@ -33,10 +33,6 @@ public class LocationManagerConnector { private final Handler handler; private final LocationManager locationManager; - private LocationListener requestLocationUpdates; - private float requestLocationUpdatesDistance; - private long requestLocationUpdatesTime; - public LocationManagerConnector(Context context, Looper looper) { this.handler = new Handler(looper); @@ -55,10 +51,7 @@ public class LocationManagerConnector { @Override @SuppressLint("MissingPermission") public void run() { - requestLocationUpdatesTime = minTime; - requestLocationUpdatesDistance = minDistance; - requestLocationUpdates = locationListener; - locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, requestLocationUpdatesTime, requestLocationUpdatesDistance, locationListener); + locationManager.requestLocationUpdates(LocationManager.GPS_PROVIDER, minTime, minDistance, locationListener); } }); } @@ -70,6 +63,5 @@ public class LocationManagerConnector { */ public void removeLocationUpdates(final LocationListener locationListener) { locationManager.removeUpdates(locationListener); - requestLocationUpdates = null; } } diff --git a/src/main/java/de/dennisguse/opentracks/services/TrackRecordingService.java b/src/main/java/de/dennisguse/opentracks/services/TrackRecordingService.java index 24292ee86..3229b2caa 100644 --- a/src/main/java/de/dennisguse/opentracks/services/TrackRecordingService.java +++ b/src/main/java/de/dennisguse/opentracks/services/TrackRecordingService.java @@ -80,13 +80,13 @@ import de.dennisguse.opentracks.util.UnitConversions; public class TrackRecordingService extends Service { // The name of extra intent property to indicate whether we want to resume a previously recorded track. - public static final String RESUME_TRACK_EXTRA_NAME = "de.dennisguse.opentracks.RESUME_TRACK"; + public static final String RESUME_TRACK_EXTRA_NAME = "RESUME_TRACK"; + private static final int NOTIFICATION_ID = 123; public static final double PAUSE_LATITUDE = 100.0; public static final double RESUME_LATITUDE = 200.0; // Anything faster than that (in meters per second) will be considered moving. - public static final double MAX_NO_MOVEMENT_SPEED = 0.224; @VisibleForTesting static final int MAX_AUTO_RESUME_TRACK_RETRY_ATTEMPTS = 3; @@ -423,7 +423,7 @@ public class TrackRecordingService extends Service { .setSmallIcon(R.drawable.ic_logo_color_24dp) .setCategory(Notification.CATEGORY_SERVICE) .setWhen(System.currentTimeMillis()); - startForeground(1, builder.build()); + startForeground(NOTIFICATION_ID, builder.build()); } /**