Fixed the bug that caused stale state to persist when opening Direct Icon commands without sheet.

Author: cryptrr

app/src/main/java/com/autosec/pie/DirectCommandActivity.kt

@@ -93,10 +93,10 @@ class DirectCommandActivity : ComponentActivity() {
                 //Get the command from the list
 
                 scope.launch {
-                    delay(250L)
+                    delay(100L)
                     if(commandId != null){
                         Timber.d("Setting command to $commandId")
-                        val success = shareReceiverViewModel.selectCommandFromDirectActivity(commandId)
+                        val success = shareReceiverViewModel.selectCommandFromDirectActivity(commandId, activity)
                     }
                 }
 

app/src/main/java/com/autosec/pie/autopieapp/presentation/screens/CommandDetailsSheet.kt

@@ -29,6 +29,7 @@ import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.sp
+import com.autosec.pie.BuildConfig
 import com.autosec.pie.R
 import com.autosec.pie.autopieapp.data.CommandModel
 import com.autosec.pie.autopieapp.data.ShareInputs
@@ -73,7 +74,7 @@ fun CommandDetailsSheet(
                 .setIcon(Icon.createWithResource(context, R.mipmap.ic_launcher))
                 .setIntent(
                     Intent(Intent.ACTION_MAIN).apply {
-                        setClassName(context, context.packageName + ".DirectCommandActivity")
+                        setClassName(context, BuildConfig.APPLICATION_ID + ".DirectCommandActivity")
                         putExtra("commandId", commandId)
                         flags = Intent.FLAG_ACTIVITY_NEW_TASK
                     }

app/src/main/java/com/autosec/pie/autopieapp/presentation/viewModels/ShareReceiverViewModel.kt

@@ -1,5 +1,6 @@
 package com.autosec.pie.autopieapp.presentation.viewModels
 
+import android.app.Activity
 import android.app.Application
 import android.content.Intent
 import android.os.Environment
@@ -170,32 +171,33 @@ class ShareReceiverViewModel(private val application1: Application) : ViewModel(
         }
     }
 
-    fun selectCommandFromDirectActivity(commandId: String): Boolean{
-        val command = shareItemsResult.value.find { it.name == commandId }
+    fun selectCommandFromDirectActivity(commandId: String, activity: Activity?): Boolean{
+        try {
+            val command = shareItemsResult.value.find { it.name == commandId }
 
-        if(command == null){
-            Timber.d("Command not found: $commandId")
-            return false
-        }
+            if(command == null){
+                Timber.d("Command not found: $commandId")
+                return false
+            }
 
-        if (command.extras?.isNotEmpty() == true) {
-            Timber.d("Opening Extras sheet for $commandId")
-            currentExtrasDetails.value =
-                Triple(true, command, ShareInputs(null, null))
-        }else{
-            onCommandClick(command, emptyList(), null) {
-                viewModelScope.launch {
-//                    isLoading = true
-//                    delay(900)
-//                    Timber.d("CLOSING THE AUTOPIE COMMANDS SHEET.")
-//                    activity?.finish()
-
-                    delay(1500L)
-                    currentExtrasDetails.value = null
+            if (command.extras?.isNotEmpty() == true) {
+                Timber.d("Opening Extras sheet for $commandId")
+                currentExtrasDetails.value =
+                    Triple(true, command, ShareInputs(null, null))
+            }else{
+                onCommandClick(command, emptyList(), null) {
+                    viewModelScope.launch {
+                        delay(1000L)
+                        currentExtrasDetails.value = null
+                        activity?.finish()
+                    }
                 }
             }
+            return true
+        }catch (e: Exception){
+            Timber.e(e)
+            return false
         }
-        return true
     }