Bugfix: a BLE sensor disconnect should only replace the aggregator.

Fixes #1799.

src/main/java/de/dennisguse/opentracks/sensors/BluetoothConnectionManager.java

@@ -148,6 +148,7 @@ public class BluetoothConnectionManager implements Driver {
             Log.w(TAG, "NONE: going to disconnect");
             if (isConnected()) {
                 disconnect();
+                observer.onRemove(sensorHandler.createEmptySensorData(null, null));
             }
             return;
         }

src/main/java/de/dennisguse/opentracks/sensors/GainManager.java

@@ -52,7 +52,11 @@ public class GainManager implements SensorConnector {
 
         String address = PreferencesUtils.getBarometerSensorAddress();
         switch (PreferencesUtils.getSensorType(address)) {
-            case NONE -> driver = null;
+            case NONE -> {
+                driver = null;
+                listener.onRemove(new AggregatorBarometer(null, null));
+                return;
+            }
             case INTERNAL -> driver = new BarometerInternal(listener);
             case REMOTE -> driver =
                     new BluetoothConnectionManager(
@@ -63,9 +67,7 @@ public class GainManager implements SensorConnector {
             default -> throw new RuntimeException("Not implemented");
         }
 
-        if (driver != null) {
+        driver.connect(context, handler, address);
-            driver.connect(context, handler, address);
-        }
     }
 
     private void onDisconnect() {

src/main/java/de/dennisguse/opentracks/sensors/SensorManager.java

@@ -28,20 +28,26 @@ public class SensorManager implements SharedPreferences.OnSharedPreferenceChange
     private final SensorDataChangedObserver listener = new SensorDataChangedObserver() {
 
         @Override
-        public void onConnect(Aggregator<?, ?> sensorData) {
+        public void onConnect(Aggregator<?, ?> aggregator) {
-            sensorDataSet.add(sensorData);
+            sensorDataSet.add(aggregator);
             observer.onChange(new SensorDataSet(sensorDataSet));
         }
 
         @Override
-        public void onChange(Raw<?> sensorData) {
+        public void onChange(Raw<?> data) {
-            sensorDataSet.update(sensorData);
+            sensorDataSet.update(data);
             observer.onChange(new SensorDataSet(sensorDataSet));
         }
 
         @Override
-        public void onDisconnect(Aggregator<?, ?> sensorData) {
+        public void onDisconnect(Aggregator<?, ?> aggregator) {
-            sensorDataSet.remove(sensorData);
+            sensorDataSet.add(aggregator);
+            observer.onChange(new SensorDataSet(sensorDataSet));
+        }
+
+        @Override
+        public void onRemove(Aggregator<?, ?> aggregator) {
+            sensorDataSet.remove(aggregator);
             observer.onChange(new SensorDataSet(sensorDataSet));
         }
     };
@@ -134,5 +140,7 @@ public class SensorManager implements SharedPreferences.OnSharedPreferenceChange
         void onChange(Raw<?> sensorData);
 
         void onDisconnect(Aggregator<?, ?> sensorData);
+
+        void onRemove(Aggregator<?, ?> sensorData);
     }
 }

src/main/java/de/dennisguse/opentracks/services/handlers/AltitudeCorrectionManager.java

@@ -30,7 +30,6 @@ public class AltitudeCorrectionManager {
 
     public void correctAltitude(Context context, TrackPoint trackPoint) {
         if (!trackPoint.hasLocation() || !trackPoint.hasAltitude()) {
-            Log.d(TAG, "No altitude correction necessary.");
             return;
         }
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.UPSIDE_DOWN_CAKE && altitudeConverter != null) {