diff --git a/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt b/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt index 5c04ad9141..716440c36b 100644 --- a/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt +++ b/app/src/main/java/com/nextcloud/client/jobs/BackgroundJobManagerImpl.kt @@ -556,7 +556,6 @@ internal class BackgroundJobManagerImpl( val tag = startFileDownloadJobTag(user, file.fileId) val data = workDataOf( - FileDownloadWorker.WORKER_ID to file.fileId.toInt(), FileDownloadWorker.ACCOUNT_NAME to user.accountName, FileDownloadWorker.FILE_REMOTE_PATH to file.remotePath, FileDownloadWorker.BEHAVIOUR to behaviour, diff --git a/app/src/main/java/com/nextcloud/client/jobs/download/FileDownloadWorker.kt b/app/src/main/java/com/nextcloud/client/jobs/download/FileDownloadWorker.kt index 025283ca6e..8981f297c1 100644 --- a/app/src/main/java/com/nextcloud/client/jobs/download/FileDownloadWorker.kt +++ b/app/src/main/java/com/nextcloud/client/jobs/download/FileDownloadWorker.kt @@ -43,7 +43,7 @@ import java.util.Vector @Suppress("LongParameterList", "TooManyFunctions") class FileDownloadWorker( - private val viewThemeUtils: ViewThemeUtils, + viewThemeUtils: ViewThemeUtils, private val accountManager: UserAccountManager, private var localBroadcastManager: LocalBroadcastManager, private val context: Context, @@ -65,7 +65,6 @@ class FileDownloadWorker( return pendingDownloads.all.any { it.value?.payload?.isMatching(accountName, fileId) == true } } - const val WORKER_ID = "WORKER_ID" const val FILE_REMOTE_PATH = "FILE_REMOTE_PATH" const val ACCOUNT_NAME = "ACCOUNT_NAME" const val BEHAVIOUR = "BEHAVIOUR" @@ -94,7 +93,12 @@ class FileDownloadWorker( private var lastPercent = 0 private val intents = FileDownloadIntents(context) - private lateinit var notificationManager: DownloadNotificationManager + private var notificationManager = DownloadNotificationManager( + SecureRandom().nextInt(), + context, + viewThemeUtils + ) + private var downloadProgressListener = FileDownloadProgressListener() private var user: User? = null @@ -103,20 +107,12 @@ class FileDownloadWorker( private var currentUserFileStorageManager: FileDataStorageManager? = null private var fileDataStorageManager: FileDataStorageManager? = null - private var workerId: Int? = null private var downloadError: FileDownloadError? = null @Suppress("TooGenericExceptionCaught") override fun doWork(): Result { return try { val requestDownloads = getRequestDownloads() - - notificationManager = - DownloadNotificationManager( - workerId ?: SecureRandom().nextInt(), - context, - viewThemeUtils - ) addAccountUpdateListener() val foregroundInfo = ForegroundServiceHelper.createWorkerForegroundInfo( @@ -170,9 +166,6 @@ class FileDownloadWorker( } private fun getRequestDownloads(): AbstractList { - workerId = inputData.keyValueMap[WORKER_ID] as Int - Log_OC.e(TAG, "FilesDownloadWorker started for $workerId") - setUser() val files = getFiles() val downloadType = getDownloadType()