mirror of
https://github.com/element-hq/synapse
synced 2024-08-19 18:30:24 +00:00
Squash room_ids list to a set() to remove duplicates - with TODO marker as I have no idea /why/ the dups are happening in the first place
This commit is contained in:
parent
3bf2b4bc92
commit
307f94dcbe
1 changed files with 5 additions and 1 deletions
|
@ -519,7 +519,11 @@ class RoomMemberHandler(BaseHandler):
|
|||
user_id=user.to_string(), membership_list=membership_list
|
||||
)
|
||||
|
||||
defer.returnValue([r.room_id for r in rooms])
|
||||
# For some reason the list of events contains duplicates
|
||||
# TODO(paul): work out why because I really don't think it should
|
||||
room_ids = set(r.room_id for r in rooms)
|
||||
|
||||
defer.returnValue(room_ids)
|
||||
|
||||
@defer.inlineCallbacks
|
||||
def _do_local_membership_update(self, event, membership, snapshot,
|
||||
|
|
Loading…
Reference in a new issue