fix default duration not disabling

Author: Razeeman

features/feature_change_category/src/main/java/com/example/util/simpletimetracker/feature_change_category/view/ChangeCategoryFragment.kt

@@ -185,7 +185,7 @@ class ChangeCategoryFragment :
     }
 
     override fun onDisable(tag: String?) {
-        viewModel.onDurationDisabled(tag)
+        viewModel.onGoalDurationDisabled(tag)
     }
 
     private fun updateUi(item: CategoryViewData) = with(binding) {

features/feature_change_goals/api/src/main/java/com/example/util/simpletimetracker/feature_change_goals/api/GoalsViewModelDelegate.kt

@@ -12,7 +12,7 @@ interface GoalsViewModelDelegate {
     fun onGoalsVisible()
     fun onNotificationsHintClick()
     fun onGoalDurationSet(tag: String?, duration: Long, anchor: Any)
-    fun onDurationDisabled(tag: String?)
+    fun onGoalDurationDisabled(tag: String?)
     fun onGoalTypeSelected(range: RecordTypeGoal.Range, position: Int)
     fun onGoalCountChange(range: RecordTypeGoal.Range, count: String)
     fun onGoalTimeClick(range: RecordTypeGoal.Range)

features/feature_change_goals/src/main/java/com/example/util/simpletimetracker/feature_change_goals/delegate/GoalsViewModelDelegateImpl.kt

@@ -55,7 +55,7 @@ class GoalsViewModelDelegateImpl @Inject constructor(
         checkExactAlarmPermissionInteractor.execute(anchor)
     }
 
-    override fun onDurationDisabled(tag: String?) {
+    override fun onGoalDurationDisabled(tag: String?) {
         if (tag !in tags) return
         onNewGoalDuration(tag, 0)
     }

features/feature_change_record_type/src/main/java/com/example/util/simpletimetracker/feature_change_record_type/viewModel/ChangeRecordTypeViewModel.kt

@@ -331,7 +331,6 @@ class ChangeRecordTypeViewModel @Inject constructor(
         DurationDialogParams(
             tag = DEFAULT_DURATION_DIALOG_TAG,
             value = DurationDialogParams.Value.DurationSeconds(newDefaultDuration),
-            hideDisableButton = true,
         ).let(router::navigate)
     }
 
@@ -343,12 +342,17 @@ class ChangeRecordTypeViewModel @Inject constructor(
 
     fun onDurationSet(tag: String?, duration: Long, anchor: Any) {
         goalsViewModelDelegate.onGoalDurationSet(tag, duration, anchor)
-        onInstantDurationSet(tag, duration)
+        onDefaultDurationSet(tag, duration)
     }
 
-    private fun onInstantDurationSet(tag: String?, duration: Long) {
+    fun onDurationDisabled(tag: String?) {
+        goalsViewModelDelegate.onGoalDurationDisabled(tag)
+        onDefaultDurationSet(tag, 0)
+    }
+
+    private fun onDefaultDurationSet(tag: String?, duration: Long) {
         if (tag != DEFAULT_DURATION_DIALOG_TAG) return
-        newDefaultDuration = duration.coerceAtLeast(1)
+        newDefaultDuration = duration
         updateAdditionalState()
     }