fix: Differentiate between stopping and destroying -> destroy service after stoppind and saving last recording information

app/src/main/java/app/myzel394/alibi/services/RecorderService.kt

@@ -55,11 +55,6 @@ abstract class RecorderService : LifecycleService() {
 
     protected open suspend fun stop() {
         recordingTimeTimer.shutdown()
-
-        NotificationManagerCompat.from(this)
-            .cancel(NotificationHelper.RECORDER_CHANNEL_NOTIFICATION_ID)
-        stopForeground(STOP_FOREGROUND_REMOVE)
-        stopSelf()
     }
 
     protected abstract fun startForegroundService()
@@ -85,6 +80,13 @@ abstract class RecorderService : LifecycleService() {
         changeState(RecorderState.RECORDING)
     }
 
+    fun destroy() {
+        NotificationManagerCompat.from(this)
+            .cancel(NotificationHelper.RECORDER_CHANNEL_NOTIFICATION_ID)
+        stopForeground(STOP_FOREGROUND_REMOVE)
+        stopSelf()
+    }
+
     override fun onBind(intent: Intent): IBinder? {
         super.onBind(intent)
         return binder

app/src/main/java/app/myzel394/alibi/ui/components/RecorderScreen/atoms/RecorderEventsHandler.kt

@@ -82,16 +82,14 @@ fun RecorderEventsHandler(
         }
     }
 
-    fun saveAsLastRecording(
+    suspend fun saveAsLastRecording(
         recorder: RecorderModel
     ) {
         if (!settings.deleteRecordingsImmediately) {
-            scope.launch {
+            dataStore.updateData {
-                dataStore.updateData {
+                it.setLastRecording(
-                    it.setLastRecording(
+                    recorder.recorderService!!.getRecordingInformation()
-                        recorder.recorderService!!.getRecordingInformation()
+                )
-                    )
-                }
             }
         }
     }
@@ -189,14 +187,24 @@ fun RecorderEventsHandler(
     // Register audio recorder events
     DisposableEffect(key1 = audioRecorder, key2 = settings) {
         audioRecorder.onRecordingSave = {
-            saveAsLastRecording(audioRecorder as RecorderModel)
+            scope.launch {
+                audioRecorder.stopRecording(context)
 
-            saveRecording(audioRecorder)
+                kotlin.runCatching {
+                    saveAsLastRecording(audioRecorder as RecorderModel)
+
+                    saveRecording(audioRecorder)
+                }
+
+                audioRecorder.destroyService(context)
+            }
         }
         audioRecorder.onError = {
-            saveAsLastRecording(audioRecorder as RecorderModel)
+            scope.launch {
+                saveAsLastRecording(audioRecorder as RecorderModel)
 
-            showRecorderError = true
+                showRecorderError = true
+            }
         }
 
         onDispose {
@@ -208,14 +216,24 @@ fun RecorderEventsHandler(
     // Register video recorder events
     DisposableEffect(key1 = videoRecorder, key2 = settings) {
         videoRecorder.onRecordingSave = {
-            saveAsLastRecording(videoRecorder as RecorderModel)
+            scope.launch {
+                videoRecorder.stopRecording(context)
 
-            saveRecording(videoRecorder)
+                kotlin.runCatching {
+                    saveAsLastRecording(videoRecorder as RecorderModel)
+
+                    saveRecording(videoRecorder)
+                }
+
+                videoRecorder.destroyService(context)
+            }
         }
         videoRecorder.onError = {
-            saveAsLastRecording(videoRecorder as RecorderModel)
+            scope.launch {
+                saveAsLastRecording(videoRecorder as RecorderModel)
 
-            showRecorderError = true
+                showRecorderError = true
+            }
         }
 
         onDispose {

app/src/main/java/app/myzel394/alibi/ui/components/RecorderScreen/organisms/AudioRecordingStatus.kt

@@ -89,10 +89,7 @@ fun AudioRecordingStatus(
                 }
             },
             onSave = {
-                scope.launch {
+                audioRecorder.onRecordingSave()
-                    audioRecorder.stopRecording(context)
-                    audioRecorder.onRecordingSave()
-                }
             }
         )
 

app/src/main/java/app/myzel394/alibi/ui/components/RecorderScreen/organisms/VideoRecordingStatus.kt

@@ -115,10 +115,7 @@ fun VideoRecordingStatus(
                     }
                 },
                 onSave = {
-                    scope.launch {
+                    videoRecorder.onRecordingSave()
-                        videoRecorder.stopRecording(context)
-                        videoRecorder.onRecordingSave()
-                    }
                 }
             )
         }

app/src/main/java/app/myzel394/alibi/ui/models/BaseRecorderModel.kt

@@ -136,14 +136,9 @@ abstract class BaseRecorderModel<S : IntervalRecorderService.Settings, I, T : In
         // TODO: Also show what camera is in use while recording
         recorderService!!.stopRecording()
 
-        val intent = Intent(context, intentClass)
         runCatching {
             context.unbindService(connection)
         }
-        runCatching {
-            context.stopService(intent)
-        }
-        reset()
     }
 
     fun pauseRecording() {
@@ -154,6 +149,16 @@ abstract class BaseRecorderModel<S : IntervalRecorderService.Settings, I, T : In
         recorderService!!.resumeRecording()
     }
 
+    fun destroyService(context: Context) {
+        recorderService!!.destroy()
+        reset()
+        val intent = Intent(context, intentClass)
+
+        runCatching {
+            context.stopService(intent)
+        }
+    }
+
     // Bind functions used to manually bind to the service if the app
     // is closed and reopened for example
     fun bindToService(context: Context) {