kiwix/kiwix-android
1
0
Fork 0
mirror of https://github.com/kiwix/kiwix-android.git synced 2025-08-03 18:56:44 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #4279 from kiwix/Fixes#4278

Browse Source 
Fixed: Application crashes on `FetchDownloadNotificationManager.getCancelNotification` method.
This commit is contained in:
Kelson 2025-04-10 09:21:00 +02:00 committed by GitHub
commit 563e06e302
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 32 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

60
core/src/main/java/org/kiwix/kiwixmobile/core/downloader/downloadManager/FetchDownloadNotificationManager.kt
View File

@ -73,6 +73,8 @@ class FetchDownloadNotificationManager @Inject constructor(
val context: Context,
private val downloadRoomDao: DownloadRoomDao
) : DefaultFetchNotificationManager(context) {
private val notificationBuilderLock = Any()
private val downloadNotificationManager: NotificationManager by lazy {
context.getSystemService(Context.NOTIFICATION_SERVICE) as NotificationManager
}
@ -258,34 +260,36 @@ class FetchDownloadNotificationManager @Inject constructor(
download: Download,
notificationBuilder: Builder
): Notification {
val downloadTitle = getDownloadNotificationTitle(download)
val notificationTitle =
runBlocking(Dispatchers.IO) {
downloadRoomDao.getEntityForFileName(downloadTitle)?.title
?: downloadTitle
}
return notificationBuilder.setPriority(NotificationCompat.PRIORITY_DEFAULT)
.setSmallIcon(android.R.drawable.stat_sys_download_done)
.setContentTitle(notificationTitle)
.setContentText(context.getString(R.string.paused_state))
// Set the ongoing true so that could not cancel the pause notification.
// However, on Android 14 and above user can cancel the notification by swipe right so we
// can't control that see https://developer.android.com/about/versions/14/behavior-changes-all#non-dismissable-notifications
.setOngoing(true)
.setGroup(download.id.toString())
.setGroupSummary(false)
.setProgress(HUNDERED, download.progress, false)
.addAction(
drawable.fetch_notification_cancel,
context.getString(R.string.cancel),
getActionPendingIntent(fetch, download, DownloadNotification.ActionType.DELETE)
)
.addAction(
drawable.fetch_notification_resume,
context.getString(R.string.notification_resume_button_text),
getActionPendingIntent(fetch, download, DownloadNotification.ActionType.RESUME)
)
.build()
synchronized(notificationBuilderLock) {
val downloadTitle = getDownloadNotificationTitle(download)
val notificationTitle =
runBlocking(Dispatchers.IO) {
downloadRoomDao.getEntityForFileName(downloadTitle)?.title
?: downloadTitle
}
return notificationBuilder.setPriority(NotificationCompat.PRIORITY_DEFAULT)
.setSmallIcon(android.R.drawable.stat_sys_download_done)
.setContentTitle(notificationTitle)
.setContentText(context.getString(R.string.paused_state))
// Set the ongoing true so that could not cancel the pause notification.
// However, on Android 14 and above user can cancel the notification by swipe right so we
// can't control that see https://developer.android.com/about/versions/14/behavior-changes-all#non-dismissable-notifications
.setOngoing(true)
.setGroup(download.id.toString())
.setGroupSummary(false)
.setProgress(HUNDERED, download.progress, false)
.addAction(
drawable.fetch_notification_cancel,
context.getString(R.string.cancel),
getActionPendingIntent(fetch, download, DownloadNotification.ActionType.DELETE)
)
.addAction(
drawable.fetch_notification_resume,
context.getString(R.string.notification_resume_button_text),
getActionPendingIntent(fetch, download, DownloadNotification.ActionType.RESUME)
)
.build()
}
}
private fun getActionPendingIntent(