Refresh only when Changes are done!

Author: fatalcoder524

app/src/main/java/com/github/capntrips/kernelflasher/MainActivity.kt

@@ -71,6 +71,10 @@ import kotlinx.serialization.ExperimentalSerializationApi
 import java.io.File
 import kotlin.system.exitProcess
 
+object SharedViewModels {
+    lateinit var mainViewModel: MainViewModel
+}
+
 @ExperimentalAnimationApi
 @ExperimentalMaterialApi
 @ExperimentalMaterial3Api
@@ -210,6 +214,7 @@ class MainActivity : ComponentActivity() {
                 MainViewModel(application, fileSystemManager, navController)
             }
             val mainViewModel = viewModel!!
+            SharedViewModels.mainViewModel = mainViewModel
 
             val context = LocalContext.current
             val dialogData = viewModel!!.updateDialogData

app/src/main/java/com/github/capntrips/kernelflasher/ui/screens/main/MainViewModel.kt

@@ -50,16 +50,23 @@ class MainViewModel(
     val hasRamoops: Boolean
 
     private val _isRefreshing: MutableState<Boolean> = mutableStateOf(true)
+    private val _isRefreshRequired = mutableStateOf(true)
     private var _error: String? = null
     private var _backups: MutableMap<String, Backup> = mutableMapOf()
 
     val isRefreshing: Boolean
         get() = _isRefreshing.value
+    val isRefreshRequired: Boolean
+        get() = _isRefreshRequired.value
     val hasError: Boolean
         get() = _error != null
     val error: String
         get() = _error!!
 
+    fun markRefreshNeeded() {
+        _isRefreshRequired.value = true
+    }
+
     data class UpdateDialogData(
         val title: String,
         val changelog: List<String>,
@@ -112,15 +119,20 @@ class MainViewModel(
 
         hasRamoops = fileSystemManager.getFile("/sys/fs/pstore/console-ramoops-0").exists()
         _isRefreshing.value = false
+        _isRefreshRequired.value = false
     }
 
     fun refresh(context: Context) {
+        if (!isRefreshRequired) return
+
         launch {
             slotA.refresh(context)
             if (isAb) {
                 slotB!!.refresh(context)
             }
             backups.refresh(context)
+
+            _isRefreshRequired.value = false
         }
     }
 

app/src/main/java/com/github/capntrips/kernelflasher/ui/screens/slot/SlotViewModel.kt

@@ -15,6 +15,7 @@ import androidx.compose.runtime.snapshots.SnapshotStateMap
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
 import androidx.navigation.NavController
+import com.github.capntrips.kernelflasher.SharedViewModels
 import com.github.capntrips.kernelflasher.common.PartitionUtil
 import com.github.capntrips.kernelflasher.common.extensions.ByteArray.toHex
 import com.github.capntrips.kernelflasher.common.extensions.ExtendedFile.inputStream
@@ -27,6 +28,7 @@ import com.topjohnwu.superuser.nio.FileSystemManager
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
+import kotlinx.serialization.ExperimentalSerializationApi
 import kotlinx.serialization.json.Json
 import java.io.File
 import java.security.DigestOutputStream
@@ -585,6 +587,7 @@ class SlotViewModel(
         Shell.cmd("chmod +rwx $file").exec()
     }
 
+    @OptIn(ExperimentalSerializationApi::class)
     @Suppress("FunctionName")
     private suspend fun _flashAk3(context: Context, type: String) {
         if (!isActive) {
@@ -620,6 +623,7 @@ class SlotViewModel(
 					showCautionDialog() // Show dialog instead of uiPrint
 				}
             }
+            SharedViewModels.mainViewModel.markRefreshNeeded()
         }
     }
 
@@ -800,6 +804,7 @@ class SlotViewModel(
                     }
                 }
             }
+            SharedViewModels.mainViewModel.markRefreshNeeded()
         }
     }
 
@@ -848,6 +853,7 @@ class SlotViewModel(
                         showCautionDialog() // Show dialog instead of uiPrint
                     }
                 }
+                SharedViewModels.mainViewModel.markRefreshNeeded()
             }
         }
     }