diff options
author | Erik Johnston <erik@matrix.org> | 2019-10-02 15:47:22 +0100 |
---|---|---|
committer | Erik Johnston <erik@matrix.org> | 2019-10-02 15:47:22 +0100 |
commit | 7a5f080f91f42fe011a1ab497acdce481187da5f (patch) | |
tree | 0a7b18f5b715c51e815ce39605fb593c2fcc56da /synapse | |
parent | Merge pull request #6152 from matrix-org/erikj/fix_room_list (diff) | |
download | synapse-7a5f080f91f42fe011a1ab497acdce481187da5f.tar.xz |
Fix appservice room list pagination
Diffstat (limited to 'synapse')
-rw-r--r-- | synapse/storage/room.py | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/synapse/storage/room.py b/synapse/storage/room.py index 9b7e31583c..70bd719521 100644 --- a/synapse/storage/room.py +++ b/synapse/storage/room.py @@ -150,6 +150,24 @@ class RoomWorkerStore(SQLBaseStore): where_clauses = [] query_args = [] + if network_tuple: + if network_tuple.appservice_id: + published_sql = """ + SELECT room_id from appservice_room_list + WHERE appservice_id = ? AND network_id = ? + """ + query_args.append(network_tuple.appservice_id) + query_args.append(network_tuple.network_id) + else: + published_sql = """ + SELECT room_id FROM rooms WHERE is_public + """ + else: + published_sql = """ + SELECT room_id FROM rooms WHERE is_public + UNION SELECT room_id from appservice_room_list + """ + # Work out the bounds if we're given them, these bounds look slightly # odd, but are designed to help query planner use indices by pulling # out a common bound. @@ -188,24 +206,6 @@ class RoomWorkerStore(SQLBaseStore): ) query_args += [search_term, search_term, search_term] - if network_tuple: - if network_tuple.appservice_id: - published_sql = """ - SELECT room_id from appservice_room_list - WHERE appservice_id = ? AND network_id = ? - """ - query_args.append(network_tuple.appservice_id) - query_args.append(network_tuple.network_id) - else: - published_sql = """ - SELECT room_id FROM rooms WHERE is_public - """ - else: - published_sql = """ - SELECT room_id FROM rooms WHERE is_public - UNION SELECT room_id from appservice_room_list - """ - where_clause = "" if where_clauses: where_clause = " AND " + " AND ".join(where_clauses) |