Merge pull request #1825 from Hafizzle/Hafiz/default-dailies-active-fix

Fix default dailies filter

Habitica/src/main/java/com/habitrpg/android/habitica/ui/fragments/tasks/TaskRecyclerViewFragment.kt

@@ -321,6 +321,8 @@ open class TaskRecyclerViewFragment : BaseFragment<FragmentRefreshRecyclerviewBi
                 .doOnNext { recyclerAdapter?.showAdventureGuide = !it.hasCompletedOnboarding }
                 .subscribe({ recyclerAdapter?.user = it }, RxErrorHandler.handleEmptyError())
         )
+
+        setPreferenceTaskFilters()
     }
 
     private fun updateTaskSubscription(ownerID: String?) {
@@ -458,26 +460,6 @@ open class TaskRecyclerViewFragment : BaseFragment<FragmentRefreshRecyclerviewBi
         }
     }
 
-    override fun onStart() {
-        super.onStart()
-        (activity as? MainActivity)?.viewModel?.user?.observeOnce(this) {
-            if (it != null) {
-                when (taskType) {
-                    TaskType.TODO -> viewModel.setActiveFilter(
-                        TaskType.TODO,
-                        Task.FILTER_ACTIVE
-                    )
-                    TaskType.DAILY -> {
-                        if (it.isValid && it.preferences?.dailyDueDefaultView == true) {
-                            viewModel.setActiveFilter(TaskType.DAILY, Task.FILTER_ACTIVE)
-                        }
-                    }
-                    else -> {}
-                }
-            }
-        }
-    }
-
     override fun onResume() {
         super.onResume()
         context?.let { recyclerAdapter?.taskDisplayMode = configManager.taskDisplayMode(it) }
@@ -496,6 +478,28 @@ open class TaskRecyclerViewFragment : BaseFragment<FragmentRefreshRecyclerviewBi
         }
     }
 
+    private fun setPreferenceTaskFilters() {
+        (activity as? MainActivity)?.viewModel?.user?.observeOnce(this) {
+            if (it != null) {
+                when (taskType) {
+                    TaskType.TODO -> viewModel.setActiveFilter(
+                        TaskType.TODO,
+                        Task.FILTER_ACTIVE
+                    )
+                    TaskType.DAILY -> {
+                        if (!viewModel.initialPreferenceFilterSet) {
+                            viewModel.initialPreferenceFilterSet = true
+                            if (it.isValid && it.preferences?.dailyDueDefaultView == true) {
+                                viewModel.setActiveFilter(TaskType.DAILY, Task.FILTER_ACTIVE)
+                            }
+                        }
+                    }
+                    else -> {}
+                }
+            }
+        }
+    }
+
     private fun openTaskForm(task: Task) {
         if (Date().time - (TasksFragment.lastTaskFormOpen?.time ?: 0) < 2000 || !task.isValid || !canEditTasks) {
             return

Habitica/src/main/java/com/habitrpg/android/habitica/ui/viewmodels/TasksViewModel.kt

@@ -49,6 +49,7 @@ class TasksViewModel : BaseViewModel() {
     val ownerID: MutableLiveData<String?> by lazy {
         MutableLiveData()
     }
+    var initialPreferenceFilterSet: Boolean = false
 
     val isPersonalBoard: Boolean
         get() {