Added: try catch for update

pakka-papad · Jan 26, 2025 · 13bdfd8 · 13bdfd8
1 parent e2e9f13
commit 13bdfd8
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 17 deletions.
diff --git a/app/src/main/java/com/github/pakka_papad/MainActivity.kt b/app/src/main/java/com/github/pakka_papad/MainActivity.kt
@@ -5,6 +5,7 @@ import android.os.Bundle
 import androidx.appcompat.app.AppCompatActivity
 import androidx.core.splashscreen.SplashScreen.Companion.installSplashScreen
 import androidx.core.view.WindowCompat
+import com.github.pakka_papad.data.ZenCrashReporter
 import com.github.pakka_papad.data.ZenPreferenceProvider
 import com.github.pakka_papad.databinding.ActivityMainBinding
 import com.google.android.play.core.appupdate.AppUpdateManager
@@ -23,6 +24,8 @@ class MainActivity : AppCompatActivity() {
 
     @Inject lateinit var appUpdateManager: AppUpdateManager
 
+    @Inject lateinit var crashReporter: ZenCrashReporter
+
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         installSplashScreen().apply {
@@ -40,14 +43,18 @@ class MainActivity : AppCompatActivity() {
     override fun onResume() {
         super.onResume()
 
-        appUpdateManager.appUpdateInfo.addOnSuccessListener {
-            if (it.updateAvailability() == UpdateAvailability.DEVELOPER_TRIGGERED_UPDATE_IN_PROGRESS){
-                appUpdateManager.startUpdateFlow(
-                    it,
-                    this,
-                    AppUpdateOptions.defaultOptions(AppUpdateType.IMMEDIATE)
-                )
+        try {
+            appUpdateManager.appUpdateInfo.addOnSuccessListener {
+                if (it.updateAvailability() == UpdateAvailability.DEVELOPER_TRIGGERED_UPDATE_IN_PROGRESS){
+                    appUpdateManager.startUpdateFlow(
+                        it,
+                        this,
+                        AppUpdateOptions.defaultOptions(AppUpdateType.IMMEDIATE)
+                    )
+                }
             }
+        } catch (e: Exception) {
+            crashReporter.logException(e)
         }
     }
 }
diff --git a/app/src/main/java/com/github/pakka_papad/settings/SettingsFragment.kt b/app/src/main/java/com/github/pakka_papad/settings/SettingsFragment.kt
@@ -22,6 +22,7 @@ import androidx.navigation.fragment.findNavController
 import com.github.pakka_papad.R
 import com.github.pakka_papad.components.Snackbar
 import com.github.pakka_papad.components.TopBarWithBackArrow
+import com.github.pakka_papad.data.ZenCrashReporter
 import com.github.pakka_papad.data.ZenPreferenceProvider
 import com.github.pakka_papad.ui.theme.ZenTheme
 import com.google.android.play.core.appupdate.AppUpdateManager
@@ -37,11 +38,11 @@ class SettingsFragment : Fragment() {
 
     private lateinit var navController: NavController
 
-    @Inject
-    lateinit var preferenceProvider: ZenPreferenceProvider
+    @Inject lateinit var preferenceProvider: ZenPreferenceProvider
 
-    @Inject
-    lateinit var appUpdateManager: AppUpdateManager
+    @Inject lateinit var appUpdateManager: AppUpdateManager
+
+    @Inject lateinit var crashReporter: ZenCrashReporter
 
     override fun onCreateView(
         inflater: LayoutInflater,
@@ -132,12 +133,16 @@ class SettingsFragment : Fragment() {
     private fun onAppUpdateClicked() {
         val appUpdateInfo = viewModel.appUpdateInfo.value
         viewModel.consumeAppUpdateInfo()
-        appUpdateInfo?.let {
-            appUpdateManager.startUpdateFlow(
-                it,
-                requireActivity(),
-                AppUpdateOptions.defaultOptions(AppUpdateType.IMMEDIATE)
-            )
+        try {
+            appUpdateInfo?.let {
+                appUpdateManager.startUpdateFlow(
+                    it,
+                    requireActivity(),
+                    AppUpdateOptions.defaultOptions(AppUpdateType.IMMEDIATE)
+                )
+            }
+        } catch (e: Exception) {
+            crashReporter.logException(e)
         }
     }
 }