sadorowo/wulkanowy-mod
1
0
Fork 0
forked from github/wulkanowy-mirror
Code Issues Pull Requests Actions Releases 5 Activity

Fix NPE when trying to remove a message from mailbox that doesn't match any student (#2090)

Browse Source
This commit is contained in:
Mikołaj Pich 2023-01-01 20:21:28 +01:00 committed by GitHub
parent 7efd106658
commit 83974b6550
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/src/main/java/io/github/wulkanowy/data/repositories/MessageRepository.kt
View File

@ -178,7 +178,7 @@ class MessageRepository @Inject constructor(
).first() ).first()
} }
suspend fun deleteMessage(student: Student, mailbox: Mailbox, message: Message) { suspend fun deleteMessage(student: Student, mailbox: Mailbox?, message: Message) {
deleteMessages(student, mailbox, listOf(message)) deleteMessages(student, mailbox, listOf(message))
} }

2
app/src/main/java/io/github/wulkanowy/ui/modules/message/preview/MessagePreviewPresenter.kt
View File

@ -186,7 +186,7 @@ class MessagePreviewPresenter @Inject constructor(
runCatching { runCatching {
val student = studentRepository.getCurrentStudent(decryptPass = true) val student = studentRepository.getCurrentStudent(decryptPass = true)
val mailbox = messageRepository.getMailboxByStudent(student) val mailbox = messageRepository.getMailboxByStudent(student)
messageRepository.deleteMessage(student, mailbox!!, message!!) messageRepository.deleteMessage(student, mailbox, message!!)
} }
.onFailure { .onFailure {
retryCallback = { onMessageDelete() } retryCallback = { onMessageDelete() }