fix pomodoro not stopping on timer deleted manually

Author: Razeeman

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

@@ -47,19 +47,20 @@ class RemoveRunningRecordMediator @Inject constructor(
             updateWidgets = updateWidgets,
             updateNotificationSwitch = updateNotificationSwitch,
         )
-        pomodoroStopInteractor.checkAndStop(runningRecord.id)
     }
 
     suspend fun remove(
         typeId: Long,
         updateWidgets: Boolean = true,
         updateNotificationSwitch: Boolean = true,
+        checkPomodoroStop: Boolean = true,
     ) {
         runningRecordInteractor.remove(typeId)
         updateExternalViewsInteractor.onRunningRecordRemove(
             typeId = typeId,
             updateWidgets = updateWidgets,
             updateNotificationSwitch = updateNotificationSwitch,
         )
+        if (checkPomodoroStop) pomodoroStopInteractor.checkAndStop(typeId)
     }
 }

features/feature_change_running_record/src/main/java/com/example/util/simpletimetracker/feature_change_running_record/viewModel/ChangeRunningRecordViewModel.kt

@@ -140,6 +140,7 @@ class ChangeRunningRecordViewModel @Inject constructor(
             typeId = extra.id,
             updateWidgets = false,
             updateNotificationSwitch = false,
+            checkPomodoroStop = extra.id != newTypeId,
         )
         addRunningRecordMediator.addAfterChange(
             typeId = newTypeId,

features/feature_dialogs/src/main/java/com/example/util/simpletimetracker/feature_dialogs/recordQuickActions/interactor/RecordQuickActionsInteractor.kt

@@ -141,6 +141,7 @@ class RecordQuickActionsInteractor @Inject constructor(
             typeId = old.id,
             updateWidgets = false,
             updateNotificationSwitch = false,
+            checkPomodoroStop = old.id != newTypeId
         )
         addRunningRecordMediator.addAfterChange(
             typeId = newTypeId,

features/feature_pomodoro/src/main/res/layout/pomodoro_fragment.xml

@@ -5,26 +5,15 @@
     android:layout_width="match_parent"
     android:layout_height="match_parent">
 
-    <androidx.cardview.widget.CardView
+    <com.google.android.material.button.MaterialButton
         android:id="@+id/btnPomodoroSettings"
-        style="@style/SettingsCard"
-        android:layout_width="44dp"
-        android:layout_height="44dp"
-        android:layout_margin="8dp"
-        android:foreground="?selectableItemBackground"
-        app:cardBackgroundColor="?appCardBackgroundColor"
-        app:layout_constraintDimensionRatio="h,1:1"
-        app:layout_constraintEnd_toEndOf="parent"
-        app:layout_constraintTop_toTopOf="parent">
-
-        <androidx.appcompat.widget.AppCompatImageView
-            android:layout_width="match_parent"
-            android:layout_height="match_parent"
-            android:scaleType="center"
-            android:tint="?appInactiveColor"
-            app:srcCompat="@drawable/tab_settings" />
-
-    </androidx.cardview.widget.CardView>
+        style="@style/ContainerRangeOptionsButton"
+        android:layout_marginEnd="8dp"
+        android:backgroundTint="?appCardBackgroundColor"
+        app:icon="@drawable/tab_settings"
+        app:iconTint="?appInactiveColor"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent" />
 
     <androidx.constraintlayout.widget.ConstraintLayout
         android:layout_width="0dp"