From b2e8666c9f46bc691d4c80a921ba011b7991e8aa Mon Sep 17 00:00:00 2001 From: Greyson Parrelli Date: Fri, 27 Mar 2026 16:09:58 -0400 Subject: [PATCH] Avoid chaining in BackupMessagesJob. --- .../securesms/jobs/BackupMessagesJob.kt | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/BackupMessagesJob.kt b/app/src/main/java/org/thoughtcrime/securesms/jobs/BackupMessagesJob.kt index 5d7d1951ba..1ac0822855 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/jobs/BackupMessagesJob.kt +++ b/app/src/main/java/org/thoughtcrime/securesms/jobs/BackupMessagesJob.kt @@ -109,15 +109,7 @@ class BackupMessagesJob private constructor( return } - val jobManager = AppDependencies.jobManager - - val chain = jobManager.startChain(BackupMessagesJob()) - - if (SignalStore.backup.optimizeStorage && SignalStore.backup.backsUpMedia) { - chain.then(OptimizeMediaJob()) - } - - chain.enqueue() + AppDependencies.jobManager.add(BackupMessagesJob()) } fun cancel() { @@ -159,6 +151,14 @@ class BackupMessagesJob private constructor( } override fun run(): Result { + val result = doWork() + if (result.isSuccess && !isCanceled && SignalStore.backup.optimizeStorage && SignalStore.backup.backsUpMedia) { + AppDependencies.jobManager.add(OptimizeMediaJob()) + } + return result + } + + private fun doWork(): Result { if (!isBackupAllowed()) { Log.d(TAG, "Skip running BackupMessagesJob.", true) return Result.success()