Sulkta-OSS/element-x-ada
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Tweak the DefaultWorkManagerScheduler.hasPendingWork logic

Browse source
This commit is contained in:
Jorge Martín 2025-12-10 13:22:39 +01:00 • committed by Jorge Martin Espinosa
parent 607b1d9023
commit cf6ae7d611
1 changed files with 9 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

11
libraries/workmanager/impl/src/main/kotlin/io/element/android/libraries/workmanager/impl/DefaultWorkManagerScheduler.kt
View file

@ -8,6 +8,7 @@
package io.element.android.libraries.workmanager.impl
import androidx.work.WorkInfo
import androidx.work.WorkManager
import dev.zacsweers.metro.AppScope
import dev.zacsweers.metro.ContributesBinding
@ -53,8 +54,14 @@ class DefaultWorkManagerScheduler(
override fun hasPendingWork(sessionId: SessionId, requestType: WorkManagerRequestType): Boolean {
val workInfos = workManager.getWorkInfosByTag(workManagerTag(sessionId, requestType)).get().orEmpty()
// It has pending work if it's periodic or it isn't but it's not finished
return workInfos.any { it.periodicityInfo != null || !it.state.isFinished }
return workInfos.any { info ->
val isPeriodic = info.periodicityInfo != null
val isCancelled = info.state == WorkInfo.State.CANCELLED
// It has pending work if:
// - It's not periodic and is not finished.
// - It's periodic and is not cancelled - since it'll be run again in a next iteration otherwise
!isPeriodic && !info.state.isFinished || isPeriodic && !isCancelled
}
}
override fun cancel(sessionId: SessionId) {