From 8dcefc59ebeadbb7770ebdaeeb82c4b6b1e5a195 Mon Sep 17 00:00:00 2001
From: Myzel394 <50424412+Myzel394@users.noreply.github.com>
Date: Thu, 17 Aug 2023 12:48:52 +0200
Subject: [PATCH] fix: Properly stop foreground service

---
 .../myzel394/alibi/ui/models/AudioRecorderModel.kt  | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/app/myzel394/alibi/ui/models/AudioRecorderModel.kt b/app/src/main/java/app/myzel394/alibi/ui/models/AudioRecorderModel.kt
index bd5aec6..e9638cf 100644
--- a/app/src/main/java/app/myzel394/alibi/ui/models/AudioRecorderModel.kt
+++ b/app/src/main/java/app/myzel394/alibi/ui/models/AudioRecorderModel.kt
@@ -37,7 +37,6 @@ class AudioRecorderModel: ViewModel() {
     val progress: Float
         get() = (recordingTime!! / recorderService!!.settings!!.maxDuration).toFloat()
 
-    private var intent: Intent? = null
     var recorderService: AudioRecorderService? = null
         private set
 
@@ -86,15 +85,13 @@ class AudioRecorderModel: ViewModel() {
     }
 
     fun startRecording(context: Context) {
-        reset()
-
         runCatching {
             context.unbindService(connection)
         }
 
-        intent = Intent(context, AudioRecorderService::class.java)
-        ContextCompat.startForegroundService(context, intent!!)
-        context.bindService(intent!!, connection, Context.BIND_AUTO_CREATE)
+        val intent = Intent(context, AudioRecorderService::class.java)
+        ContextCompat.startForegroundService(context, intent)
+        context.bindService(intent, connection, Context.BIND_AUTO_CREATE)
     }
 
     fun stopRecording(context: Context, saveAsLastRecording: Boolean = true) {
@@ -104,9 +101,11 @@ class AudioRecorderModel: ViewModel() {
 
         runCatching {
             context.unbindService(connection)
-            context.stopService(intent)
         }
 
+        val intent = Intent(context, AudioRecorderService::class.java)
+        context.stopService(intent)
+
         reset()
     }