Browse Source

Merge pull request #3810 from matrix-org/erikj/send_tags_down_sync_on_join

Send existing room tags down sync on join
Amber Brown 5 years ago
parent
commit
9a5ea511b5
2 changed files with 14 additions and 0 deletions
  1. 1 0
      changelog.d/3810.bugfix
  2. 13 0
      synapse/handlers/sync.py

+ 1 - 0
changelog.d/3810.bugfix

@@ -0,0 +1 @@
+Fix existing room tags not coming down sync when joining a room

+ 13 - 0
synapse/handlers/sync.py

@@ -1575,6 +1575,19 @@ class SyncHandler(object):
             newly_joined_room=newly_joined,
         )
 
+        # When we join the room (or the client requests full_state), we should
+        # send down any existing tags. Usually the user won't have tags in a
+        # newly joined room, unless either a) they've joined before or b) the
+        # tag was added by synapse e.g. for server notice rooms.
+        if full_state:
+            user_id = sync_result_builder.sync_config.user.to_string()
+            tags = yield self.store.get_tags_for_room(user_id, room_id)
+
+            # If there aren't any tags, don't send the empty tags list down
+            # sync
+            if not tags:
+                tags = None
+
         account_data_events = []
         if tags is not None:
             account_data_events.append({