move external views update to one place

Author: Razeeman

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

@@ -1,15 +1,11 @@
 package com.example.util.simpletimetracker.domain.interactor
 
 import com.example.util.simpletimetracker.domain.model.Record
-import com.example.util.simpletimetracker.domain.model.WidgetType
 import javax.inject.Inject
 
 class AddRecordMediator @Inject constructor(
     private val recordInteractor: RecordInteractor,
-    private val notificationTypeInteractor: NotificationTypeInteractor,
-    private val notificationActivitySwitchInteractor: NotificationActivitySwitchInteractor,
-    private val notificationGoalTimeInteractor: NotificationGoalTimeInteractor,
-    private val widgetInteractor: WidgetInteractor,
+    private val externalViewsInteractor: UpdateExternalViewsInteractor,
 ) {
 
     suspend fun add(
@@ -27,12 +23,9 @@ class AddRecordMediator @Inject constructor(
         typeId: Long,
         updateNotificationSwitch: Boolean = true,
     ) {
-        notificationTypeInteractor.checkAndShow(typeId)
-        if (updateNotificationSwitch) {
-            notificationActivitySwitchInteractor.updateNotification()
-        }
-        notificationGoalTimeInteractor.checkAndReschedule(listOf(typeId))
-        widgetInteractor.updateWidgets(listOf(WidgetType.STATISTICS_CHART))
-        widgetInteractor.updateSingleWidgets(typeIds = listOf(typeId))
+        externalViewsInteractor.onRecordAddOrChange(
+            typeId = typeId,
+            updateNotificationSwitch = updateNotificationSwitch,
+        )
     }
 }

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

@@ -1,6 +1,5 @@
 package com.example.util.simpletimetracker.domain.interactor
 
-import com.example.util.simpletimetracker.domain.extension.orZero
 import com.example.util.simpletimetracker.domain.model.Range
 import com.example.util.simpletimetracker.domain.model.Record
 import com.example.util.simpletimetracker.domain.model.RecordType
@@ -17,18 +16,12 @@ class AddRunningRecordMediator @Inject constructor(
     private val recordTypeInteractor: RecordTypeInteractor,
     private val addRecordMediator: AddRecordMediator,
     private val recordTypeToDefaultTagInteractor: RecordTypeToDefaultTagInteractor,
-    private val notificationTypeInteractor: NotificationTypeInteractor,
-    private val notificationActivitySwitchInteractor: NotificationActivitySwitchInteractor,
-    private val notificationInactivityInteractor: NotificationInactivityInteractor,
-    private val notificationActivityInteractor: NotificationActivityInteractor,
-    private val notificationGoalTimeInteractor: NotificationGoalTimeInteractor,
     private val notificationGoalCountInteractor: NotificationGoalCountInteractor,
-    private val widgetInteractor: WidgetInteractor,
-    private val wearInteractor: WearInteractor,
     private val activityStartedStoppedBroadcastInteractor: ActivityStartedStoppedBroadcastInteractor,
     private val shouldShowTagSelectionInteractor: ShouldShowTagSelectionInteractor,
     private val pomodoroStartInteractor: PomodoroStartInteractor,
     private val complexRuleProcessActionInteractor: ComplexRuleProcessActionInteractor,
+    private val updateExternalViewsInteractor: UpdateExternalViewsInteractor,
 ) {
 
     /**
@@ -150,24 +143,18 @@ class AddRunningRecordMediator @Inject constructor(
         params: StartParams,
     ) {
         if (runningRecordInteractor.get(params.typeId) == null && params.typeId > 0L) {
-            RunningRecord(
+            val data = RunningRecord(
                 id = params.typeId,
                 timeStarted = params.timeStarted,
                 comment = params.comment,
                 tagIds = params.tagIds,
-            ).let {
-                runningRecordInteractor.add(it)
-                notificationTypeInteractor.checkAndShow(params.typeId)
-                if (params.updateNotificationSwitch) {
-                    notificationActivitySwitchInteractor.updateNotification()
-                }
-                notificationInactivityInteractor.cancel()
-                // Schedule only on first activity start.
-                if (runningRecordInteractor.getAll().size == 1) notificationActivityInteractor.checkAndSchedule()
-                notificationGoalTimeInteractor.checkAndReschedule(listOf(params.typeId))
-                widgetInteractor.updateWidgets()
-                wearInteractor.update()
-            }
+            )
+
+            runningRecordInteractor.add(data)
+            updateExternalViewsInteractor.onRunningRecordAdd(
+                typeId = params.typeId,
+                updateNotificationSwitch = params.updateNotificationSwitch,
+            )
         }
     }
 

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

@@ -8,11 +8,7 @@ import javax.inject.Inject
 
 class BackupInteractor @Inject constructor(
     private val backupRepo: BackupRepo,
-    private val widgetInteractor: WidgetInteractor,
-    private val notificationTypeInteractor: NotificationTypeInteractor,
-    private val notificationActivitySwitchInteractor: NotificationActivitySwitchInteractor,
-    private val notificationGoalTimeInteractor: NotificationGoalTimeInteractor,
-    private val wearInteractor: WearInteractor,
+    private val externalViewsInteractor: UpdateExternalViewsInteractor,
 ) {
 
     suspend fun saveBackupFile(
@@ -46,10 +42,6 @@ class BackupInteractor @Inject constructor(
     }
 
     suspend fun doAfterRestore() {
-        notificationTypeInteractor.updateNotifications()
-        notificationActivitySwitchInteractor.updateNotification()
-        notificationGoalTimeInteractor.checkAndReschedule()
-        widgetInteractor.updateWidgets()
-        wearInteractor.update()
+        externalViewsInteractor.onBackupRestore()
     }
 }

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

@@ -1,16 +1,14 @@
 package com.example.util.simpletimetracker.domain.interactor
 
 import com.example.util.simpletimetracker.domain.model.Range
-import com.example.util.simpletimetracker.domain.model.WidgetType
 import com.example.util.simpletimetracker.domain.resolver.CsvRepo
 import com.example.util.simpletimetracker.domain.resolver.ResultCode
 import javax.inject.Inject
 
 class CsvExportInteractor @Inject constructor(
     private val csvRepo: CsvRepo,
     private val runningRecordInteractor: RunningRecordInteractor,
-    private val widgetInteractor: WidgetInteractor,
-    private val notificationGoalTimeInteractor: NotificationGoalTimeInteractor,
+    private val externalViewsInteractor: UpdateExternalViewsInteractor,
 ) {
 
     suspend fun saveCsvFile(uriString: String, range: Range?): ResultCode {
@@ -21,9 +19,7 @@ class CsvExportInteractor @Inject constructor(
         val resultCode = csvRepo.importCsvFile(uriString)
 
         val runningRecords = runningRecordInteractor.getAll()
-        notificationGoalTimeInteractor.checkAndReschedule(runningRecords.map { it.id })
-        widgetInteractor.updateWidgets(listOf(WidgetType.STATISTICS_CHART))
-        widgetInteractor.updateWidgets(listOf(WidgetType.RECORD_TYPE))
+        externalViewsInteractor.onCsvImport(runningRecords.map { it.id })
 
         return resultCode
     }

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

@@ -1,14 +1,10 @@
 package com.example.util.simpletimetracker.domain.interactor
 
-import com.example.util.simpletimetracker.domain.model.WidgetType
 import javax.inject.Inject
 
 class RemoveRecordMediator @Inject constructor(
     private val recordInteractor: RecordInteractor,
-    private val notificationTypeInteractor: NotificationTypeInteractor,
-    private val notificationActivitySwitchInteractor: NotificationActivitySwitchInteractor,
-    private val notificationGoalTimeInteractor: NotificationGoalTimeInteractor,
-    private val widgetInteractor: WidgetInteractor,
+    private val externalViewsInteractor: UpdateExternalViewsInteractor,
 ) {
 
     suspend fun remove(recordId: Long, typeId: Long) {
@@ -17,10 +13,6 @@ class RemoveRecordMediator @Inject constructor(
     }
 
     suspend fun doAfterRemove(typeId: Long) {
-        notificationTypeInteractor.checkAndShow(typeId)
-        notificationActivitySwitchInteractor.updateNotification()
-        notificationGoalTimeInteractor.checkAndReschedule(listOf(typeId))
-        widgetInteractor.updateWidgets(listOf(WidgetType.STATISTICS_CHART))
-        widgetInteractor.updateSingleWidgets(typeIds = listOf(typeId))
+        externalViewsInteractor.onRecordRemove(typeId)
     }
 }

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

@@ -4,7 +4,7 @@ import javax.inject.Inject
 
 class RemoveRecordTagMediator @Inject constructor(
     private val recordTagInteractor: RecordTagInteractor,
-    private val wearInteractor: WearInteractor,
+    private val externalViewsInteractor: UpdateExternalViewsInteractor,
 ) {
 
     suspend fun remove(tagId: Long) {
@@ -13,6 +13,6 @@ class RemoveRecordTagMediator @Inject constructor(
     }
 
     private suspend fun doAfterRemove() {
-        wearInteractor.update()
+        externalViewsInteractor.onTagRemove()
     }
 }

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

@@ -1,16 +1,10 @@
 package com.example.util.simpletimetracker.domain.interactor
 
-import com.example.util.simpletimetracker.domain.model.RecordTypeGoal
-import com.example.util.simpletimetracker.domain.model.RunningRecord
-import com.example.util.simpletimetracker.domain.model.WidgetType
 import javax.inject.Inject
 
 class RemoveRecordTypeMediator @Inject constructor(
     private val recordTypeInteractor: RecordTypeInteractor,
-    private val notificationGoalTimeInteractor: NotificationGoalTimeInteractor,
-    private val widgetInteractor: WidgetInteractor,
-    private val wearInteractor: WearInteractor,
-    private val runningRecordInteractor: RunningRecordInteractor,
+    private val externalViewsInteractor: UpdateExternalViewsInteractor,
 ) {
 
     suspend fun remove(typeId: Long) {
@@ -19,10 +13,6 @@ class RemoveRecordTypeMediator @Inject constructor(
     }
 
     private suspend fun doAfterRemove(typeId: Long) {
-        val runningRecordIds = runningRecordInteractor.getAll().map(RunningRecord::id)
-        notificationGoalTimeInteractor.cancel(RecordTypeGoal.IdData.Type(typeId))
-        notificationGoalTimeInteractor.checkAndReschedule(runningRecordIds + typeId)
-        widgetInteractor.updateWidgets(listOf(WidgetType.STATISTICS_CHART))
-        wearInteractor.update()
+        externalViewsInteractor.onTypeRemove(typeId)
     }
 }

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

@@ -7,16 +7,10 @@ import javax.inject.Inject
 class RemoveRunningRecordMediator @Inject constructor(
     private val recordInteractor: RecordInteractor,
     private val runningRecordInteractor: RunningRecordInteractor,
-    private val notificationTypeInteractor: NotificationTypeInteractor,
-    private val notificationActivitySwitchInteractor: NotificationActivitySwitchInteractor,
-    private val notificationInactivityInteractor: NotificationInactivityInteractor,
-    private val notificationActivityInteractor: NotificationActivityInteractor,
-    private val notificationGoalTimeInteractor: NotificationGoalTimeInteractor,
-    private val widgetInteractor: WidgetInteractor,
-    private val wearInteractor: WearInteractor,
     private val prefsInteractor: PrefsInteractor,
     private val activityStartedStoppedBroadcastInteractor: ActivityStartedStoppedBroadcastInteractor,
     private val pomodoroStopInteractor: PomodoroStopInteractor,
+    private val updateExternalViewsInteractor: UpdateExternalViewsInteractor,
 ) {
 
     suspend fun removeWithRecordAdd(
@@ -51,18 +45,10 @@ class RemoveRunningRecordMediator @Inject constructor(
         updateNotificationSwitch: Boolean = true,
     ) {
         runningRecordInteractor.remove(typeId)
-        notificationTypeInteractor.checkAndHide(typeId)
-        if (updateNotificationSwitch) {
-            notificationActivitySwitchInteractor.updateNotification()
-        }
-        notificationInactivityInteractor.checkAndSchedule()
-        // Cancel if no activity tracked.
-        val runningRecordIds = runningRecordInteractor.getAll().map { it.id }
-        if (runningRecordIds.isEmpty()) notificationActivityInteractor.cancel()
-        notificationGoalTimeInteractor.checkAndReschedule(runningRecordIds + typeId)
-        if (updateWidgets) {
-            widgetInteractor.updateWidgets()
-            wearInteractor.update()
-        }
+        updateExternalViewsInteractor.onRunningRecordRemove(
+            typeId = typeId,
+            updateWidgets = updateWidgets,
+            updateNotificationSwitch = updateNotificationSwitch
+        )
     }
 }