Merge branch 'krille/replace-old-statevent' into 'main'

fix: use originServerTs to check if state event is old

See merge request famedly/company/frontend/famedlysdk!894
This commit is contained in:
Krille Fear 2021-11-15 11:33:12 +00:00
commit 0be1576465
3 changed files with 2 additions and 7 deletions

View File

@ -67,8 +67,6 @@ abstract class DatabaseApi {
Future<Event?> getEventById(String eventId, Room room);
bool eventIsKnown(String eventId, String roomId);
Future<void> forgetRoom(String roomId);
Future<void> clearCache();

View File

@ -371,10 +371,6 @@ class FamedlySdkHiveDatabase extends DatabaseApi {
return Event.fromJson(convertToJson(raw), room);
}
@override
bool eventIsKnown(String eventId, String roomId) =>
_eventsBox.keys.contains(MultiKey(roomId, eventId).toString());
/// Loads a whole list of events at once from the store for a specific room
Future<List<Event>> _getEventsByIds(List<String> eventIds, Room room) =>
Future.wait(eventIds

View File

@ -203,7 +203,8 @@ class Room {
final prevEvent = getState(state.type, stateKey);
if (prevEvent != null &&
prevEvent.eventId != state.eventId &&
client.database?.eventIsKnown(state.eventId, roomId) == true) {
prevEvent.originServerTs.millisecondsSinceEpoch >
state.originServerTs.millisecondsSinceEpoch) {
return;
}