Modify group room association API to allow modification of is_public

also includes renamings to make things more consistent.
This commit is contained in:
Luke Barnard 2017-10-31 17:04:28 +00:00
parent c31a7c3ff6
commit 20fe347906
6 changed files with 25 additions and 19 deletions

View file

@ -531,7 +531,7 @@ class GroupsServerHandler(object):
})
@defer.inlineCallbacks
def add_room_to_group(self, group_id, requester_user_id, room_id, content):
def update_room_group_association(self, group_id, requester_user_id, room_id, content):
"""Add room to group
"""
RoomID.from_string(room_id) # Ensure valid room id
@ -542,19 +542,19 @@ class GroupsServerHandler(object):
is_public = _parse_visibility_from_contents(content)
yield self.store.add_room_to_group(group_id, room_id, is_public=is_public)
yield self.store.update_room_group_association(group_id, room_id, is_public=is_public)
defer.returnValue({})
@defer.inlineCallbacks
def remove_room_from_group(self, group_id, requester_user_id, room_id):
def delete_room_group_association(self, group_id, requester_user_id, room_id):
"""Remove room from group
"""
yield self.check_group_is_ours(
group_id, requester_user_id, and_exists=True, and_is_admin=requester_user_id
)
yield self.store.remove_room_from_group(group_id, room_id)
yield self.store.delete_room_group_association(group_id, room_id)
defer.returnValue({})