Release: v4.16.0

Problem was that the TrackRecordingService was not started as a ForegroundService anymore.

Fixes #1904.

build.gradle

@@ -6,7 +6,7 @@ buildscript {
         mavenCentral()
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:8.7.2'
+        classpath 'com.android.tools.build:gradle:8.7.3'
     }
 }
 
@@ -64,8 +64,8 @@ android {
 
     defaultConfig {
         applicationId "de.dennisguse.opentracks"
-        versionCode 5938
-        versionName "v4.15.0"
+        versionCode 5941
+        versionName "v4.16.0"
 
         buildConfigField "String", "VERSION_NAME_FULL", "\"${getVersionName()}\""
 

fastlane/metadata/android/en-US/changelogs/5941.txt

@@ -0,0 +1,5 @@
+v4.16.0: OpenTracks
+
+Bugfix:
+- PublicAPI: starting a recording didn't record no data
+

src/androidTest/java/de/dennisguse/opentracks/publicapi/PublicApiTest.java

@@ -31,13 +31,7 @@ public class PublicApiTest {
 
     private final Context context = ApplicationProvider.getApplicationContext();
 
-    @Test
-    public void StartTest() {
-        PreferencesUtils.setBoolean(R.string.publicapi_enabled_key, true);
-
-        context.startActivity(IntentUtils.newIntent(context, StartRecording.class));
-    }
-
+    //NOTE: this doesn't check if the TrackRecordingService was started in foreground.
     @Test
     public void StartStopTest() throws InterruptedException {
         PreferencesUtils.setBoolean(R.string.publicapi_enabled_key, true);

src/main/java/de/dennisguse/opentracks/publicapi/AbstractAPIActivity.java

@@ -33,11 +33,15 @@ public abstract class AbstractAPIActivity extends AppCompatActivity {
     protected void onCreate(@Nullable Bundle savedInstanceState) {
         SplashScreen splashScreen = SplashScreen.installSplashScreen(this);
         super.onCreate(savedInstanceState);
-        splashScreen.setKeepOnScreenCondition(() -> true );
+        splashScreen.setKeepOnScreenCondition(() -> true);
 
         if (PreferencesUtils.isPublicAPIenabled()) {
             Log.i(TAG, "Received and trying to execute requested action.");
-            TrackRecordingServiceConnection.execute(this, serviceConnectedCallback);
+            if (requiresForeground()) {
+                TrackRecordingServiceConnection.executeForeground(this, serviceConnectedCallback);
+            } else {
+                TrackRecordingServiceConnection.execute(this, serviceConnectedCallback);
+            }
         } else {
             Toast.makeText(this, getString(R.string.settings_public_api_disabled_toast), Toast.LENGTH_LONG).show();
             Log.w(TAG, "Public API is disabled; ignoring request.");
@@ -48,4 +52,8 @@ public abstract class AbstractAPIActivity extends AppCompatActivity {
     protected abstract void execute(TrackRecordingService service);
 
     protected abstract boolean isPostExecuteStopService();
+
+    protected boolean requiresForeground() {
+        return false;
+    }
 }

src/main/java/de/dennisguse/opentracks/publicapi/StartRecording.java

@@ -62,4 +62,9 @@ public class StartRecording extends AbstractAPIActivity {
     protected boolean isPostExecuteStopService() {
         return false;
     }
+
+    @Override
+    protected boolean requiresForeground() {
+        return true;
+    }
 }