add time started calculation from previous record if adding record on…

… today
kantahrek · Nov 26, 2023 · 7fb6d25 · 7fb6d25
1 parent 9c4848d
commit 7fb6d25
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 12 deletions.
diff --git a/...m/example/util/simpletimetracker/feature_change_record/viewModel/ChangeRecordViewModel.kt b/...m/example/util/simpletimetracker/feature_change_record/viewModel/ChangeRecordViewModel.kt
@@ -106,7 +106,7 @@ class ChangeRecordViewModel @Inject constructor(
             timeStarted = newTimeStarted,
             timeEnded = newTimeEnded,
             comment = newComment,
-            tagIds = newCategoryIds
+            tagIds = newCategoryIds,
         ).let {
             addRecordMediator.add(it)
             if (newTypeId != originalTypeId) {
@@ -143,7 +143,7 @@ class ChangeRecordViewModel @Inject constructor(
             timeStarted = newTimeStarted,
             timeEnded = newTimeEnded,
             comment = newComment,
-            tagIds = newCategoryIds
+            tagIds = newCategoryIds,
         ).let {
             addRecordMediator.add(it)
         }
@@ -166,10 +166,23 @@ class ChangeRecordViewModel @Inject constructor(
         super.onTimeStartedChanged()
     }
 
-    private fun getInitialDate(daysFromToday: Int): Long {
+    private fun getInitialTimeEnded(daysFromToday: Int): Long {
         return timeMapper.toTimestampShifted(daysFromToday, RangeLength.Day)
     }
 
+    private suspend fun getInitialTimeStarted(daysFromToday: Int): Long {
+        val default = newTimeEnded - ONE_HOUR
+
+        return if (daysFromToday == 0) {
+            recordInteractor.getPrev(newTimeEnded, limit = 1)
+                .firstOrNull()
+                ?.timeEnded
+                ?: default
+        } else {
+            default
+        }
+    }
+
     override suspend fun updatePreview() {
         (record as MutableLiveData).value = loadPreviewViewData()
     }
@@ -190,8 +203,9 @@ class ChangeRecordViewModel @Inject constructor(
                 newTimeEnded = (extra as ChangeRecordParams.Untracked).timeEnded
             }
             is ChangeRecordParams.New -> {
-                newTimeEnded = getInitialDate((extra as ChangeRecordParams.New).daysFromToday)
-                newTimeStarted = newTimeEnded - ONE_HOUR
+                val daysFromToday = (extra as ChangeRecordParams.New).daysFromToday
+                newTimeEnded = getInitialTimeEnded(daysFromToday)
+                newTimeStarted = getInitialTimeStarted(daysFromToday)
             }
         }
         newTimeSplit = newTimeStarted
@@ -207,7 +221,7 @@ class ChangeRecordViewModel @Inject constructor(
             timeStarted = newTimeStarted,
             timeEnded = newTimeEnded,
             comment = newComment,
-            tagIds = newCategoryIds
+            tagIds = newCategoryIds,
         )
 
         return changeRecordViewDataInteractor.getPreviewViewData(record)

diff --git a/...xample/util/simpletimetracker/feature_change_record_type/view/ChangeRecordTypeFragment.kt b/...xample/util/simpletimetracker/feature_change_record_type/view/ChangeRecordTypeFragment.kt
@@ -2,9 +2,7 @@ package com.example.util.simpletimetracker.feature_change_record_type.view
 
 import android.os.Bundle
 import android.view.LayoutInflater
-import android.view.View
 import android.view.ViewGroup
-import androidx.cardview.widget.CardView
 import androidx.core.view.isVisible
 import androidx.core.view.updatePadding
 import androidx.core.widget.doAfterTextChanged
@@ -21,7 +19,6 @@ import com.example.util.simpletimetracker.core.extension.showKeyboard
 import com.example.util.simpletimetracker.core.extension.toViewData
 import com.example.util.simpletimetracker.core.repo.DeviceRepo
 import com.example.util.simpletimetracker.core.utils.fragmentArgumentDelegate
-import com.example.util.simpletimetracker.core.utils.setChooserColor
 import com.example.util.simpletimetracker.core.view.buttonsRowView.ButtonsRowViewData
 import com.example.util.simpletimetracker.domain.model.IconEmojiType
 import com.example.util.simpletimetracker.domain.model.IconType
@@ -57,8 +54,6 @@ import com.example.util.simpletimetracker.feature_change_record_type.viewModel.C
 import com.example.util.simpletimetracker.feature_views.extension.addOnScrollListenerAdapter
 import com.example.util.simpletimetracker.feature_views.extension.dpToPx
 import com.example.util.simpletimetracker.feature_views.extension.pxToDp
-import com.example.util.simpletimetracker.feature_views.extension.rotateDown
-import com.example.util.simpletimetracker.feature_views.extension.rotateUp
 import com.example.util.simpletimetracker.feature_views.extension.setOnClick
 import com.example.util.simpletimetracker.feature_views.extension.setSpanSizeLookup
 import com.example.util.simpletimetracker.feature_views.extension.visible

diff --git a/...til/simpletimetracker/feature_change_record_type/viewData/ChangeRecordTypeChooserState.kt b/...til/simpletimetracker/feature_change_record_type/viewData/ChangeRecordTypeChooserState.kt
@@ -11,6 +11,6 @@ data class ChangeRecordTypeChooserState(
         object Icon : State
         object Category : State
         object GoalTime : State
-        object Type: State
+        object Type : State
     }
 }