Add a showProgressCatching method to avoid crashing when showing prog…

…ress This happens quite often even though it seems that we try and do everything to avoid it
Infomaniak · Jun 27, 2024 · c5576ed · c5576ed
1 parent 8e458a0
commit c5576ed
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 7 deletions.
diff --git a/Core b/Core
diff --git a/app/src/main/java/com/infomaniak/mail/ui/alertDialogs/BaseAlertDialog.kt b/app/src/main/java/com/infomaniak/mail/ui/alertDialogs/BaseAlertDialog.kt
@@ -28,7 +28,7 @@ import com.google.android.material.button.MaterialButton
 import com.infomaniak.lib.core.utils.Utils
 import com.infomaniak.lib.core.utils.hideProgress
 import com.infomaniak.lib.core.utils.initProgress
-import com.infomaniak.lib.core.utils.showProgress
+import com.infomaniak.lib.core.utils.showProgressCatching
 import com.infomaniak.mail.R
 
 abstract class BaseAlertDialog(activityContext: Context) : DefaultLifecycleObserver {
@@ -45,7 +45,7 @@ abstract class BaseAlertDialog(activityContext: Context) : DefaultLifecycleObser
     fun startLoading() {
         alertDialog.setCancelable(false)
         negativeButton.isEnabled = false
-        Utils.createRefreshTimer(onTimerFinish = positiveButton::showProgress).start()
+        Utils.createRefreshTimer(onTimerFinish = positiveButton::showProgressCatching).start()
     }
 
     fun resetLoadingAndDismiss() = with(alertDialog) {

diff --git a/app/src/main/java/com/infomaniak/mail/ui/login/LoginFragment.kt b/app/src/main/java/com/infomaniak/mail/ui/login/LoginFragment.kt
@@ -198,7 +198,7 @@ class LoginFragment : Fragment() {
     }
 
     private fun startProgress() {
-        binding.connectButton.showProgress(getCurrentOnPrimary())
+        binding.connectButton.showProgressCatching(getCurrentOnPrimary())
     }
 
     private fun getCurrentOnPrimary(): Int? = introViewModel.updatedAccentColor.value?.first?.getOnPrimary(requireContext())

diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/InvalidPasswordFragment.kt b/app/src/main/java/com/infomaniak/mail/ui/main/InvalidPasswordFragment.kt
@@ -142,6 +142,6 @@ class InvalidPasswordFragment : Fragment() {
     }
 
     private fun startProgress() {
-        binding.confirmButton.showProgress()
+        binding.confirmButton.showProgressCatching()
     }
 }
diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/menu/RestoreEmailsBottomSheetDialog.kt b/app/src/main/java/com/infomaniak/mail/ui/main/menu/RestoreEmailsBottomSheetDialog.kt
@@ -126,6 +126,6 @@ class RestoreEmailsBottomSheetDialog : BottomSheetDialogFragment() {
     }
 
     private fun startProgress() {
-        binding.restoreMailsButton.showProgress()
+        binding.restoreMailsButton.showProgressCatching()
     }
 }
diff --git a/app/src/main/java/com/infomaniak/mail/ui/main/user/AttachMailboxFragment.kt b/app/src/main/java/com/infomaniak/mail/ui/main/user/AttachMailboxFragment.kt
@@ -131,6 +131,6 @@ class AttachMailboxFragment : Fragment() {
     }
 
     private fun startProgress() {
-        binding.attachMailboxButton.showProgress()
+        binding.attachMailboxButton.showProgressCatching()
     }
 }
+1 −1		BugTracker/src/main/java/com/infomaniak/lib/bugtracker/BugTrackerActivity.kt
+15 −4		src/main/java/com/infomaniak/lib/core/utils/Extensions.kt