fix: Use where and map instead of a loop and a removeWhere

This commit is contained in:
Marcel 2022-01-17 22:06:29 +00:00 committed by Nicolas Werner
parent 33b1302fc2
commit 1b11efc9eb
1 changed files with 6 additions and 7 deletions

View File

@ -1158,16 +1158,15 @@ class Room {
Membership.invite, Membership.invite,
Membership.knock, Membership.knock,
]]) { ]]) {
final userList = <User>[];
final members = states[EventTypes.RoomMember]; final members = states[EventTypes.RoomMember];
if (members != null) { if (members != null) {
for (final entry in members.entries) { return members.entries
final state = entry.value; .where((entry) => entry.value.type == EventTypes.RoomMember)
if (state.type == EventTypes.RoomMember) userList.add(state.asUser); .map((entry) => entry.value.asUser)
} .where((user) => membershipFilter.contains(user.membership))
.toList();
} }
userList.removeWhere((u) => !membershipFilter.contains(u.membership)); return <User>[];
return userList;
} }
bool _requestedParticipants = false; bool _requestedParticipants = false;