Make code more readable

- Make if/else more consistent
- Add comment on findAndAddRoom()
This commit is contained in:
R Midhun Suresh
2025-03-02 17:18:17 +05:30
parent 35a9cbe3ff
commit 44a867ffc0

View File

@@ -138,12 +138,18 @@ export class RoomListStoreV3Class extends AsyncStoreWithClient<EmptyObject> {
if (!payload.isLiveEvent || !payload.isLiveUnfilteredRoomTimelineEvent || !payload.room) return;
const roomId = payload.event.getRoomId();
const tryAdd = (): boolean => {
const findAndAddRoom = (): boolean => {
const room = this.matrixClient?.getRoom(roomId);
if (room) this.addRoomAndEmit(room);
return !!room;
};
if (!tryAdd()) setTimeout(tryAdd, 100);
// We'll try finding the room associated with this event.
// If we can't find the room, we'll try again after 100ms.
if (!findAndAddRoom()) {
logger.warn(`Live timeline event ${payload.event.getId()} received without associated room`);
logger.warn(`Queuing failed room update for retry as a result.`);
setTimeout(findAndAddRoom, 100);
}
break;
}
@@ -157,11 +163,9 @@ export class RoomListStoreV3Class extends AsyncStoreWithClient<EmptyObject> {
const predecessor = roomState.findPredecessor(this.msc3946ProcessDynamicPredecessor);
if (predecessor) {
const prevRoom = this.matrixClient?.getRoom(predecessor.roomId);
if (prevRoom) {
this.roomSkipList.removeRoom(prevRoom);
} else {
logger.warn(`Unable to find predecessor room with id ${predecessor.roomId}`);
}
if (prevRoom) this.roomSkipList.removeRoom(prevRoom);
else logger.warn(`Unable to find predecessor room with id ${predecessor.roomId}`);
}
}
this.addRoomAndEmit(payload.room);