Skip to content

Commit

Permalink
Fix joined_rooms/joined_room_ids usage (#17208)
Browse files Browse the repository at this point in the history
This change was introduced in
#17203

But then #17207 was reverted
which brought back usage `joined_rooms` that needed to be updated.
Wasn't caught because `develop` wasn't up to date before merging.
  • Loading branch information
MadLittleMods authored May 16, 2024
1 parent 52a6495 commit fe07995
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions changelog.d/17208.misc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Rename to be obvious: `joined_rooms` -> `joined_room_ids`.
2 changes: 1 addition & 1 deletion synapse/handlers/sync.py
Original file line number Diff line number Diff line change
Expand Up @@ -1875,7 +1875,7 @@ async def _generate_sync_entry_for_device_list(
# or if the changed user is the syncing user (as we always
# want to include device list updates of their own devices).
if user_id == changed_user_id or any(
rid in joined_rooms for rid in entries
rid in joined_room_ids for rid in entries
):
users_that_have_changed.add(changed_user_id)
else:
Expand Down

0 comments on commit fe07995

Please sign in to comment.