filippodb
/
SchildiChat-android
mirror of https://github.com/SchildiChat/SchildiChat-android.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[TMP] Update some readmarker debugging 

Change-Id: Ia1de156f39622d90cc1433f0b16776bd00e2a462
This commit is contained in:
SpiritCroc 2022-03-11 10:04:40 +01:00
parent 12e4853c59
commit 6aabb38c3b
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
vector/src/main/java/im/vector/app/features/home/room/detail/TimelineFragment.kt
View File

@ -2039,6 +2039,7 @@ class TimelineFragment @Inject constructor(
} }
override fun onLoadMore(direction: Timeline.Direction) { override fun onLoadMore(direction: Timeline.Direction) {
Timber.i("ReadMarker debug: on load more $direction")
timelineViewModel.handle(RoomDetailAction.LoadMoreTimelineEvents(direction)) timelineViewModel.handle(RoomDetailAction.LoadMoreTimelineEvents(direction))
} }

6
vector/src/main/java/im/vector/app/features/home/room/detail/TimelineViewModel.kt
View File

@ -638,6 +638,7 @@ class TimelineViewModel @AssistedInject constructor(
if (trackUnreadMessages.getAndSet(false)) { if (trackUnreadMessages.getAndSet(false)) {
mostRecentDisplayedEvent?.root?.eventId?.also { mostRecentDisplayedEvent?.root?.eventId?.also {
session.coroutineScope.launch(NonCancellable) { session.coroutineScope.launch(NonCancellable) {
Timber.i("ReadMarker debug: set RM and RR to $it")
tryOrNull { room.setReadMarker(it) } tryOrNull { room.setReadMarker(it) }
if (loadRoomAtFirstUnread()) { if (loadRoomAtFirstUnread()) {
tryOrNull { room.setReadReceipt(it) } tryOrNull { room.setReadReceipt(it) }
@ -1127,6 +1128,7 @@ class TimelineViewModel @AssistedInject constructor(
} }
} }
.setOnEach { .setOnEach {
Timber.i("ReadMarker debug: update unreadState = $it")
copy(unreadState = it) copy(unreadState = it)
} }
} }
@ -1156,11 +1158,13 @@ class TimelineViewModel @AssistedInject constructor(
if (!isFromMe) { if (!isFromMe) {
return UnreadState.HasUnread(eventId) return UnreadState.HasUnread(eventId)
} }
Timber.i("ReadMarker debug: hasNoUnread / firstDisplayableEventIndex: $firstDisplayableEventIndex / " +
"latest previewable from summary ${roomSummary.latestPreviewableOriginalContentEvent?.eventId} - ${timeline.getIndexOfEvent(roomSummary.latestPreviewableOriginalContentEvent?.eventId)} / " +
"event-0 ${events.getOrNull(0)?.eventId}")
} }
Timber.i("ReadMarker debug: hasNoUnread / firstDisplayableEventIndex: $firstDisplayableEventIndex / " + Timber.i("ReadMarker debug: hasNoUnread / firstDisplayableEventIndex: $firstDisplayableEventIndex / " +
"latest previewable from summary ${roomSummary.latestPreviewableOriginalContentEvent?.eventId} - ${timeline.getIndexOfEvent(roomSummary.latestPreviewableOriginalContentEvent?.eventId)} / " + "latest previewable from summary ${roomSummary.latestPreviewableOriginalContentEvent?.eventId} - ${timeline.getIndexOfEvent(roomSummary.latestPreviewableOriginalContentEvent?.eventId)} / " +
"event-0 ${events.getOrNull(0)?.eventId}") "event-0 ${events.getOrNull(0)?.eventId}")
// TODO: if we return HasNoUnread, we need to be sure, since later HasUnread will be ignored in distinctUntilChanged. However, when loading the room at the last read directly, this doesn't always work
return UnreadState.HasNoUnread return UnreadState.HasNoUnread
} }