Merge pull request #48 from infinum/fix/trigger-init-not-working-prop…

…erly-on-first-start Fix: Trigger active value not correct in some cases
infinum · Mar 6, 2024 · 6877def · 6877def
2 parents 4b5123a + d4741a6
commit 6877def
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/.../src/main/kotlin/com/infinum/sentinel/data/models/memory/triggers/shared/SensorTrigger.kt b/.../src/main/kotlin/com/infinum/sentinel/data/models/memory/triggers/shared/SensorTrigger.kt
@@ -24,7 +24,6 @@ internal abstract class SensorTrigger(
         sensorManager = (context.getSystemService(Context.SENSOR_SERVICE) as? SensorManager)
         sensorManager?.let {
             registerSensor(it)
-            this.active = true
         } ?: run {
             this.active = false
         }
@@ -34,7 +33,6 @@ internal abstract class SensorTrigger(
         queue?.clear()
         unregisterSensor()
         sensorManager = null
-        this.active = false
     }
 
     override fun onSensorChanged(event: SensorEvent) {

diff --git a/sentinel/src/main/kotlin/com/infinum/sentinel/domain/triggers/TriggersRepository.kt b/sentinel/src/main/kotlin/com/infinum/sentinel/domain/triggers/TriggersRepository.kt
@@ -19,6 +19,7 @@ internal class TriggersRepository(
     override suspend fun save(input: TriggerParameters) =
         input.entity?.let {
             dao.save(it)
+            updateCache(it)
         } ?: error("Cannot save null entity")
 
     override fun load(input: TriggerParameters): Flow<List<TriggerEntity>> =