Does some of the service cleanup in a delayed manner

musicbeeremote · Dec 11, 2020 · 9bc2b68 · 9bc2b68
1 parent aea9f24
commit 9bc2b68
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/app/src/main/kotlin/com/kelsos/mbrc/controller/RemoteService.kt b/app/src/main/kotlin/com/kelsos/mbrc/controller/RemoteService.kt
@@ -102,20 +102,21 @@ class RemoteService : Service() {
     return super.onStartCommand(intent, flags, startId)
   }
 
-
   override fun onDestroy() {
     super.onDestroy()
     SERVICE_STOPPING = true;
     stopForeground(true)
     this.unregisterReceiver(receiver)
-    remoteController.executeCommand(MessageEvent(UserInputEventType.TerminateConnection))
-    CommandRegistration.unregister(remoteController)
-    threadPoolExecutor?.shutdownNow()
-    Toothpick.closeScope(this)
-
-    SERVICE_STOPPING = false
-    SERVICE_RUNNING = false
-    Timber.d("Background Service::Destroyed")
+    handler.postDelayed({
+      remoteController.executeCommand(MessageEvent(UserInputEventType.TerminateConnection))
+      CommandRegistration.unregister(remoteController)
+      threadPoolExecutor?.shutdownNow()
+      Toothpick.closeScope(this)
+
+      SERVICE_STOPPING = false
+      SERVICE_RUNNING = false
+      Timber.d("Background Service::Destroyed")
+    }, 150)
   }
 
   private inner class ControllerBinder : Binder() {

diff --git a/app/src/main/kotlin/com/kelsos/mbrc/services/SocketService.kt b/app/src/main/kotlin/com/kelsos/mbrc/services/SocketService.kt
@@ -93,6 +93,7 @@ constructor(
     Timber.v("Received action ${SocketAction.name(action)}")
     when (action) {
       RESET -> {
+        shouldStop = false;
         resetState()
         startSocket()
         cleanupSocket()
@@ -127,7 +128,6 @@ constructor(
 
   private fun resetState() {
     connecting = false
-    shouldStop = false
     numOfRetries = 0
   }