Rename to more clear get_insertion_event_id_by_batch_id (MSC2716) (#…

…11244) `get_insertion_event_by_batch_id` -> `get_insertion_event_id_by_batch_id` Split out from #11114
matrix-org · Nov 9, 2021 · 84f235a · 84f235a
1 parent 4ee71b9
commit 84f235a
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/changelog.d/11244.misc b/changelog.d/11244.misc
@@ -0,0 +1 @@
+Fix [MSC2716](https://github.com/matrix-org/matrix-doc/pull/2716) historical messages backfilling in random order on remote homeservers.
diff --git a/synapse/handlers/message.py b/synapse/handlers/message.py
@@ -1507,7 +1507,7 @@ async def persist_and_notify_client_event(
                 conflicting_insertion_event_id = None
                 if next_batch_id:
                     conflicting_insertion_event_id = (
-                        await self.store.get_insertion_event_by_batch_id(
+                        await self.store.get_insertion_event_id_by_batch_id(
                             event.room_id, next_batch_id
                         )
                     )

diff --git a/synapse/rest/client/room_batch.py b/synapse/rest/client/room_batch.py
@@ -112,7 +112,7 @@ async def on_POST(
         # and have the batch connected.
         if batch_id_from_query:
             corresponding_insertion_event_id = (
-                await self.store.get_insertion_event_by_batch_id(
+                await self.store.get_insertion_event_id_by_batch_id(
                     room_id, batch_id_from_query
                 )
             )

diff --git a/synapse/storage/databases/main/room_batch.py b/synapse/storage/databases/main/room_batch.py
@@ -18,7 +18,7 @@
 
 
 class RoomBatchStore(SQLBaseStore):
-    async def get_insertion_event_by_batch_id(
+    async def get_insertion_event_id_by_batch_id(
         self, room_id: str, batch_id: str
     ) -> Optional[str]:
         """Retrieve a insertion event ID.