disable multiselect on tab change

Author: Razeeman

core/src/main/java/com/example/util/simpletimetracker/core/sharedViewModel/MainTabsViewModel.kt

@@ -2,15 +2,19 @@ package com.example.util.simpletimetracker.core.sharedViewModel
 
 import androidx.lifecycle.LiveData
 import androidx.lifecycle.MutableLiveData
+import androidx.lifecycle.viewModelScope
 import com.example.util.simpletimetracker.core.base.BaseViewModel
 import com.example.util.simpletimetracker.core.extension.lazySuspend
 import com.example.util.simpletimetracker.core.extension.set
 import com.example.util.simpletimetracker.core.model.NavigationTab
 import com.example.util.simpletimetracker.domain.prefs.interactor.PrefsInteractor
+import com.example.util.simpletimetracker.domain.record.interactor.RecordsContainerMultiselectInteractor
+import kotlinx.coroutines.launch
 import javax.inject.Inject
 
 class MainTabsViewModel @Inject constructor(
     private val prefsInteractor: PrefsInteractor,
+    private val recordsContainerMultiselectInteractor: RecordsContainerMultiselectInteractor,
 ) : BaseViewModel() {
 
     val tabReselected: LiveData<NavigationTab?> = MutableLiveData()
@@ -21,6 +25,12 @@ class MainTabsViewModel @Inject constructor(
         tabReselected.set(tab)
     }
 
+    fun onTabUnselected(tab: NavigationTab) = viewModelScope.launch {
+        if (tab == NavigationTab.Records) {
+            recordsContainerMultiselectInteractor.disable()
+        }
+    }
+
     fun onHandled() {
         tabReselected.set(null)
     }

core/src/main/java/com/example/util/simpletimetracker/core/sharedViewModel/RemoveRecordViewModel.kt

@@ -46,7 +46,7 @@ class RemoveRecordViewModel @Inject constructor(
         val typeIds = removedRecords.map(Record::typeId).distinct()
         removeRecordMediator.remove(
             recordIds = removedRecords.map { it.id },
-            typeIds = typeIds
+            typeIds = typeIds,
         )
 
         val removedRecordsCount = removedRecords.size

domain/src/main/java/com/example/util/simpletimetracker/domain/record/interactor/RemoveRecordMediator.kt

@@ -11,7 +11,7 @@ class RemoveRecordMediator @Inject constructor(
     suspend fun remove(recordId: Long, typeId: Long) {
         remove(
             recordIds = listOf(recordId),
-            typeIds = listOf(typeId)
+            typeIds = listOf(typeId),
         )
     }
 

features/feature_dialogs/src/main/java/com/example/util/simpletimetracker/feature_dialogs/recordQuickActions/viewModel/RecordQuickActionsViewModel.kt

@@ -265,7 +265,6 @@ class RecordQuickActionsViewModel @Inject constructor(
     }
 
     private suspend fun onMultiselect() {
-        // TODO MULTI cancel on tab change?
         if (recordsContainerMultiselectInteractor.isEnabled) {
             recordsContainerMultiselectInteractor.disable()
         } else {

features/feature_main/src/main/java/com/example/util/simpletimetracker/feature_main/view/MainFragment.kt

@@ -111,6 +111,9 @@ class MainFragment : BaseFragment<Binding>() {
 
                 override fun onTabUnselected(tab: TabLayout.Tab?) {
                     tab?.icon?.colorFilter = unselectedColorFilter
+                    tab?.position
+                        ?.let(mainTabsProvider::mapPositionToTab)
+                        ?.let(mainTabsViewModel::onTabUnselected)
                 }
 
                 override fun onTabSelected(tab: TabLayout.Tab?) {