element-android/changelog.d/5528.bugfix at baefec80a0af207a4b3a21b4e738bf06f00118b8 - mirrors/element-android - Sadium Git

mirrors/element-android

mirror of https://github.com/element-hq/element-android synced 2024-12-18 07:12:47 +03:00

SpiritCroc cd45248f40 Fix modifying the wrong events in TimelineChunk

I was observing cases where builtEvents[modificationIndex] was not
having the same eventId as the udpatedEntity in handleDatabaseChangeSet.

In particular, I observed both cases that
- there was no item in the list yet with the same eventId as the updated
  one
- there was an item with the same eventId already in the list, but at a
  different position.

Whenever this happened, the timeline would render missing, duplicated,
or swapped messages in the timeline.

Instead of relying on the modificationIndex to be the same for both the
change set and builtEvents, look up the proper index by eventId.

2022-03-13 08:19:47 +01:00

1 line

54 B

Text

Raw Blame History

Fix cases of missing, swapped, or duplicated messages