Rename to more clear get_insertion_event_id_by_batch_id (MSC2716)

changelog.d/11244.misc

@@ -0,0 +1 @@
+Rename MSC2716 related function to be more clear about the input and output (`get_insertion_event_id_by_batch_id`).

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
                         )
                     )

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
                 )
             )

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.