Small rework code to avoid dead code.
This commit is contained in:
parent
ec459165db
commit
9fe9d88dc8
1 changed files with 7 additions and 13 deletions
|
|
@ -113,7 +113,7 @@ class NotifiableEventResolver @Inject constructor(
|
|||
soundName = null,
|
||||
isRedacted = false,
|
||||
isUpdated = false,
|
||||
description = descriptionFromRoomMembershipContent(content, isDirect) ?: return null,
|
||||
description = descriptionFromRoomMembershipInvite(isDirect),
|
||||
type = null, // TODO check if type is needed anymore
|
||||
title = null, // TODO check if title is needed anymore
|
||||
)
|
||||
|
|
@ -226,19 +226,13 @@ class NotifiableEventResolver @Inject constructor(
|
|||
}
|
||||
}
|
||||
|
||||
private fun descriptionFromRoomMembershipContent(
|
||||
content: NotificationContent.StateEvent.RoomMemberContent,
|
||||
private fun descriptionFromRoomMembershipInvite(
|
||||
isDirectRoom: Boolean
|
||||
): String? {
|
||||
return when (content.membershipState) {
|
||||
RoomMembershipState.INVITE -> {
|
||||
if (isDirectRoom) {
|
||||
stringProvider.getString(R.string.notification_invite_body)
|
||||
} else {
|
||||
stringProvider.getString(R.string.notification_room_invite_body)
|
||||
}
|
||||
}
|
||||
else -> null
|
||||
): String {
|
||||
return if (isDirectRoom) {
|
||||
stringProvider.getString(R.string.notification_invite_body)
|
||||
} else {
|
||||
stringProvider.getString(R.string.notification_room_invite_body)
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue