wbrawner/thunderbird-android
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Perform NotificationActionService actions in background thread

Browse source 
Updating notifications involves disk I/O.
This commit is contained in:
cketti 2022-12-01 15:39:01 +01:00
parent b862a271d1
commit 71d6a4f52e
1 changed files with 23 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
app/core/src/main/java/com/fsck/k9/notification/NotificationActionService.kt
View file

@ -11,22 +11,43 @@ import com.fsck.k9.controller.MessageReference
import com.fsck.k9.controller.MessageReferenceHelper
import com.fsck.k9.controller.MessagingController
import com.fsck.k9.mail.Flag
import kotlinx.coroutines.CoroutineScope
import kotlinx.coroutines.Dispatchers
import kotlinx.coroutines.launch
import org.koin.android.ext.android.inject
import org.koin.core.qualifier.named
import timber.log.Timber
class NotificationActionService : Service() {
private val preferences: Preferences by inject()
private val messagingController: MessagingController by inject()
private val coroutineScope: CoroutineScope by inject(named("AppCoroutineScope"))
override fun onStartCommand(intent: Intent, flags: Int, startId: Int): Int {
Timber.i("NotificationActionService started with startId = %d", startId)
val accountUuid = intent.getStringExtra(EXTRA_ACCOUNT_UUID) ?: error("Missing account UUID")
startHandleCommand(intent)
return START_NOT_STICKY
}
private fun startHandleCommand(intent: Intent) {
coroutineScope.launch(Dispatchers.IO) {
handleCommand(intent)
}
}
private fun handleCommand(intent: Intent) {
val accountUuid = intent.getStringExtra(EXTRA_ACCOUNT_UUID)
if (accountUuid == null) {
Timber.w("Missing account UUID.")
return
}
val account = preferences.getAccount(accountUuid)
if (account == null) {
Timber.w("Could not find account for notification action.")
return START_NOT_STICKY
return
}
when (intent.action) {
@ -38,8 +59,6 @@ class NotificationActionService : Service() {
}
cancelNotifications(intent, account)
return START_NOT_STICKY
}
override fun onBind(intent: Intent): IBinder? {