Merge pull request #85 from 05nelsonm/mn/bug-fix/background-manager-rebind

BugFix: background manager rebind

Author: 05nelsonm

topl-service/src/main/java/io/matthewnelson/topl_service/lifecycle/BackgroundManager.kt

@@ -320,14 +320,16 @@ class BackgroundManager internal constructor(
     private fun applicationMovedToBackground() {
         taskIsInForeground = false
         if (!ServiceActionProcessor.wasLastAcceptedServiceActionStop()) {
-            // System automatically unbinds when app is sent to the background. This prevents
-            // it so that we maintain a started, bound service.
-            BaseService.bindService(
-                BaseService.getAppContext(),
-                serviceClass,
-                bindServiceFlag = bindServiceFlag
-            )
-            TorServiceConnection.serviceBinder?.executeBackgroundPolicyJob(policy, executionDelay)
+            TorServiceConnection.serviceBinder?.let { binder ->
+                // System automatically unbinds when app is sent to the background. This prevents
+                // it so that we maintain a started, bound service.
+                BaseService.bindService(
+                    BaseService.getAppContext(),
+                    serviceClass,
+                    bindServiceFlag = bindServiceFlag
+                )
+                binder.executeBackgroundPolicyJob(policy, executionDelay)
+            }
         }
     }
 }

topl-service/src/main/java/io/matthewnelson/topl_service/service/components/binding/BaseServiceBinder.kt

@@ -66,10 +66,7 @@
 * */
 package io.matthewnelson.topl_service.service.components.binding
 
-import android.app.Activity
-import android.app.Application
 import android.os.Binder
-import android.os.Bundle
 import io.matthewnelson.topl_core.broadcaster.BroadcastLogger
 import io.matthewnelson.topl_service.service.BaseService
 import io.matthewnelson.topl_service.lifecycle.BackgroundManager
@@ -78,9 +75,7 @@ import io.matthewnelson.topl_service.service.components.actions.ServiceActions.S
 import io.matthewnelson.topl_service.util.ServiceConsts.BackgroundPolicy
 import kotlinx.coroutines.Job
 import kotlinx.coroutines.delay
-import kotlinx.coroutines.isActive
 import kotlinx.coroutines.launch
-import net.freehaven.tor.control.TorControlCommands
 
 
 internal abstract class BaseServiceBinder(private val torService: BaseService): Binder() {

topl-service/src/main/java/io/matthewnelson/topl_service/util/ServiceConsts.kt

@@ -69,7 +69,6 @@ package io.matthewnelson.topl_service.util
 import androidx.annotation.IntDef
 import androidx.annotation.StringDef
 import io.matthewnelson.topl_core_base.BaseConsts
-import io.matthewnelson.topl_service.service.components.actions.ServiceActions
 
 abstract class ServiceConsts: BaseConsts() {