Prevent updating reminders in no permission case

wordpress-mobile · Apr 6, 2023 · fb0ece3 · fb0ece3
1 parent 63431aa
commit fb0ece3
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/...ss/src/main/java/org/wordpress/android/ui/bloggingreminders/BloggingRemindersViewModel.kt b/...ss/src/main/java/org/wordpress/android/ui/bloggingreminders/BloggingRemindersViewModel.kt
@@ -203,17 +203,17 @@ class BloggingRemindersViewModel @Inject constructor(
         analyticsTracker.trackPrimaryButtonPressed(Screen.SELECTION)
         if (bloggingRemindersModel != null) {
             launch {
+                val daysCount = bloggingRemindersModel.enabledDays.size
+                if (!checkPermission()) {
+                    // There is no permission
+                    return@launch
+                }
                 bloggingRemindersStore.updateBloggingReminders(
                     mapper.toDomainModel(
                         bloggingRemindersModel
                     )
                 )
-                val daysCount = bloggingRemindersModel.enabledDays.size
                 if (daysCount > 0) {
-                    if (!checkPermission()) {
-                        // There is no permission
-                        return@launch
-                    }
                     reminderScheduler.schedule(
                         bloggingRemindersModel.siteId,
                         bloggingRemindersModel.hour,