Merge branch 'MTRNord/perf-opt-3' into 'main'

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

See merge request famedly/company/frontend/famedlysdk!937
This commit is contained in:
Nicolas Werner 2022-01-17 22:06:29 +00:00
commit 2205ffb084
1 changed files with 6 additions and 7 deletions

View File

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