Add credentials error notification

Signed-off-by: alperozturk <[email protected]>
nextcloud · Jan 8, 2024 · 25b6804 · 25b6804
1 parent 6083552
commit 25b6804
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/app/src/main/java/com/nextcloud/client/files/downloader/DownloadNotificationManager.kt b/app/src/main/java/com/nextcloud/client/files/downloader/DownloadNotificationManager.kt
@@ -120,7 +120,7 @@ class DownloadNotificationManager(
         }, 2000)
     }
 
-    private fun updateNotificationText(text: String) {
+    fun updateNotificationText(text: String) {
         notificationBuilder.run {
             setContentText(text)
             notificationManager.notify(id, this.build())

diff --git a/app/src/main/java/com/nextcloud/client/files/downloader/FileDownloadWorker.kt b/app/src/main/java/com/nextcloud/client/files/downloader/FileDownloadWorker.kt
@@ -151,9 +151,9 @@ class FileDownloadWorker(
     override fun onStopped() {
         Log_OC.e(TAG, "FilesDownloadWorker stopped")
 
-        removePendingDownload(currentDownload?.user?.accountName)
-        cancelAllDownloads()
         notificationManager.dismissNotification()
+        cancelAllDownloads()
+        removePendingDownload(currentDownload?.user?.accountName)
         setIdleWorkerState()
 
         super.onStopped()
@@ -419,6 +419,7 @@ class FileDownloadWorker(
             prepareForResult()
 
             if (needsToUpdateCredentials) {
+                updateNotificationText(context.getString(R.string.downloader_download_failed_credentials_error))
                 setContentIntent(
                     intents.credentialContentIntent(download.user),
                     PendingIntent.FLAG_ONE_SHOT or PendingIntent.FLAG_IMMUTABLE