Merge pull request #41 from jorre127/bugfix/fix-service-launching-background

fix service being launched in the background

Author: vanlooverenkoen

android/build.gradle

@@ -32,6 +32,7 @@ android {
     }
     defaultConfig {
         minSdkVersion 16
+        targetSdkVersion 33
     }
     lintOptions {
         disable 'InvalidPackage'
@@ -42,4 +43,5 @@ dependencies {
     implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
     implementation 'com.google.android.gms:play-services-location:20.0.0'
     implementation 'androidx.localbroadcastmanager:localbroadcastmanager:1.1.0'
+    implementation "androidx.core:core-ktx:1.9.0"
 }

android/src/main/kotlin/com/icapps/background_location_tracker/service/LocationUpdatesService.kt

@@ -23,6 +23,10 @@ import com.icapps.background_location_tracker.utils.ActivityCounter
 import com.icapps.background_location_tracker.utils.Logger
 import com.icapps.background_location_tracker.utils.NotificationUtil
 import com.icapps.background_location_tracker.utils.SharedPrefsUtil
+import java.io.PrintWriter
+import java.io.StringWriter
+
+private const val timeOut = 24 * 60 * 60 * 1000L /*24 hours max */
 
 internal class LocationUpdatesService : Service() {
     private val binder: IBinder = LocalBinder()
@@ -131,13 +135,23 @@ internal class LocationUpdatesService : Service() {
         // Called when the last client (MainActivity in case of this sample) unbinds from this
         // service. If this method is called due to a configuration change in MainActivity, we
         // do nothing. Otherwise, we make this service a foreground service.
-        if (!changingConfiguration && SharedPrefsUtil.isTracking(this)) {
-            Logger.debug(TAG, "Starting foreground service")
-            if (wakeLock?.isHeld != true) {
-                wakeLock?.acquire(24 * 60 * 60 * 1000L)
+
+        try {
+            if (!changingConfiguration && SharedPrefsUtil.isTracking(this)) {
+                Logger.debug(TAG, "Starting foreground service")
+                if (wakeLock?.isHeld != true) {
+                    wakeLock?.acquire(timeOut)
+                }
+                NotificationUtil.startForeground(this, location)
             }
-            NotificationUtil.startForeground(this, location)
+        } catch(e:Throwable) {
+            val sw = StringWriter()
+            val pw = PrintWriter(sw)
+            e.printStackTrace(pw)
+            pw.flush()
+            Logger.error(sw.toString(),"onUnbind failed to execute");
         }
+
         return true // Ensures onRebind() is called when a client re-binds.
     }
 
@@ -153,7 +167,7 @@ internal class LocationUpdatesService : Service() {
      * [SecurityException].
      */
     fun startTracking() {
-        wakeLock?.acquire(24 * 60 * 60 * 1000L /*24 hours max */)
+        wakeLock?.acquire(timeOut)
 
         Logger.debug(TAG, "Requesting location updates")
         SharedPrefsUtil.saveIsTracking(this, true)

android/src/main/kotlin/com/icapps/background_location_tracker/utils/NotificationUtil.kt

@@ -78,6 +78,7 @@ internal object NotificationUtil {
                 .setContentTitle(title)
                 .setContentText(text)
                 .setContentIntent(clickPendingIntent)
+                .setForegroundServiceBehavior(NotificationCompat.FOREGROUND_SERVICE_IMMEDIATE)
         if (SharedPrefsUtil.isCancelTrackingActionEnabled(context)) {
             builder.addAction(0, SharedPrefsUtil.getCancelTrackingActionText(context), cancelTrackingIntent)
         }

example/pubspec.lock

@@ -21,7 +21,7 @@ packages:
       path: ".."
       relative: true
     source: path
-    version: "1.2.0"
+    version: "1.3.0"
   boolean_selector:
     dependency: transitive
     description:

pubspec.yaml

@@ -1,7 +1,7 @@
 name: background_location_tracker
 description: A Flutter plugin that allows you to track the background location for Android & iOS
 repository: https://github.com/icapps/flutter-background-location-tracker
-version: 1.3.0
+version: 1.3.1
 
 environment:
   sdk: ">=2.12.0 <3.0.0"