Do not clear unread counts when fetching room members
This messes with our "open without reading" functionality. Change-Id: Ie1bf84aa79b3bc0c18efd7325f817aa227b8e748
This commit is contained in:
parent
128b532b51
commit
f50b21d9b7
|
@ -126,7 +126,7 @@ internal class DefaultLoadRoomMembersTask @Inject constructor(
|
||||||
roomMemberEventHandler.handle(realm, roomId, roomMemberEvent)
|
roomMemberEventHandler.handle(realm, roomId, roomMemberEvent)
|
||||||
}
|
}
|
||||||
roomEntity.membersLoadStatus = RoomMembersLoadStatusType.LOADED
|
roomEntity.membersLoadStatus = RoomMembersLoadStatusType.LOADED
|
||||||
roomSummaryUpdater.update(realm, roomId, updateMembers = true)
|
roomSummaryUpdater.update(realm, roomId, updateMembers = true, updateCounts = false)
|
||||||
}
|
}
|
||||||
|
|
||||||
if (cryptoSessionInfoProvider.isRoomEncrypted(roomId)) {
|
if (cryptoSessionInfoProvider.isRoomEncrypted(roomId)) {
|
||||||
|
|
|
@ -87,7 +87,8 @@ internal class RoomSummaryUpdater @Inject constructor(
|
||||||
unreadNotifications: RoomSyncUnreadNotifications? = null,
|
unreadNotifications: RoomSyncUnreadNotifications? = null,
|
||||||
unreadCount: Int? = null,
|
unreadCount: Int? = null,
|
||||||
updateMembers: Boolean = false,
|
updateMembers: Boolean = false,
|
||||||
inviterId: String? = null) {
|
inviterId: String? = null,
|
||||||
|
updateCounts: Boolean = true) {
|
||||||
val roomSummaryEntity = RoomSummaryEntity.getOrCreate(realm, roomId)
|
val roomSummaryEntity = RoomSummaryEntity.getOrCreate(realm, roomId)
|
||||||
if (roomSummary != null) {
|
if (roomSummary != null) {
|
||||||
if (roomSummary.heroes.isNotEmpty()) {
|
if (roomSummary.heroes.isNotEmpty()) {
|
||||||
|
@ -101,11 +102,13 @@ internal class RoomSummaryUpdater @Inject constructor(
|
||||||
roomSummaryEntity.joinedMembersCount = roomSummary.joinedMembersCount
|
roomSummaryEntity.joinedMembersCount = roomSummary.joinedMembersCount
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
roomSummaryEntity.highlightCount = unreadNotifications?.highlightCount ?: 0
|
if (updateCounts) {
|
||||||
roomSummaryEntity.notificationCount = unreadNotifications?.notificationCount ?: 0
|
roomSummaryEntity.highlightCount = unreadNotifications?.highlightCount ?: 0
|
||||||
roomSummaryEntity.unreadCount = unreadCount
|
roomSummaryEntity.notificationCount = unreadNotifications?.notificationCount ?: 0
|
||||||
roomSummaryEntity.aggregatedNotificationCount = roomSummaryEntity.notificationCount
|
roomSummaryEntity.unreadCount = unreadCount
|
||||||
roomSummaryEntity.aggregatedUnreadCount = roomSummaryEntity.safeUnreadCount()
|
roomSummaryEntity.aggregatedNotificationCount = roomSummaryEntity.notificationCount
|
||||||
|
roomSummaryEntity.aggregatedUnreadCount = roomSummaryEntity.safeUnreadCount()
|
||||||
|
}
|
||||||
|
|
||||||
if (membership != null) {
|
if (membership != null) {
|
||||||
roomSummaryEntity.membership = membership
|
roomSummaryEntity.membership = membership
|
||||||
|
|
Loading…
Reference in New Issue