Revert heroes order changes

element-hq · MadLittleMods · Jul 11, 2024 · Jul 8, 2024 · Jul 8, 2024 · Jul 8, 2024
commit 868dcdca07ada33d697f525ea08fea5bc8653f00
diff --git a/synapse/storage/databases/main/roommember.py b/synapse/storage/databases/main/roommember.py
@@ -279,19 +279,8 @@ def _get_users_in_room_with_profiles(
 
     @cached(max_entries=100000)  # type: ignore[synapse-@cached-mutable]
     async def get_room_summary(self, room_id: str) -> Mapping[str, MemberSummary]:
-        """
-        Get the details of a room roughly suitable for use by the room
+        """Get the details of a room roughly suitable for use by the room
         summary extension to /sync. Useful when lazy loading room members.
-
-        Returns the total count of members in the room by membership type, and a
-        truncated list of members (the heroes). This will be the first 6 members of the
-        room:
-        - We want 5 heroes plus 1, in case one of them is the
-        calling user.
-        - They are ordered by `stream_ordering`, which are joined or
-        invited. When no joined or invited members are available, this also includes
-        banned and left users.
-
         Args:
             room_id: The room ID to query
         Returns:
@@ -319,28 +308,23 @@ def _get_room_summary_txn(
             for count, membership in txn:
                 res.setdefault(membership, MemberSummary([], count))
 
-            # Order by membership (joins -> invites -> leave (former insiders) -> everything else
-            # including knocks since they are outsiders), then by `stream_ordering` so
-            # the first members in the room show up first and to make the sort stable
-            # (consistent heroes).
-            #
-            # Note: rejected events will have a null membership field, so we we manually
-            # filter them out.
+            # we order by membership and then fairly arbitrarily by event_id so
+            # heroes are consistent
+            # Note, rejected events will have a null membership field, so
+            # we we manually filter them out.
             sql = """
                 SELECT state_key, membership, event_id
                 FROM current_state_events
                 WHERE type = 'm.room.member' AND room_id = ?
                     AND membership IS NOT NULL
                 ORDER BY
-                    CASE membership WHEN ? THEN 1 WHEN ? THEN 2 WHEN ? THEN 3 ELSE 4 END ASC,
-                    event_stream_ordering ASC
+                    CASE membership WHEN ? THEN 1 WHEN ? THEN 2 ELSE 3 END ASC,
+                    event_id ASC
                 LIMIT ?
             """
 
             # 6 is 5 (number of heroes) plus 1, in case one of them is the calling user.
-            txn.execute(
-                sql, (room_id, Membership.JOIN, Membership.INVITE, Membership.LEAVE, 6)
-            )
+            txn.execute(sql, (room_id, Membership.JOIN, Membership.INVITE, 6))
             for user_id, membership, event_id in txn:
                 summary = res[membership]
                 # we will always have a summary for this membership type at this