Check users/rooms are in group before adding to summary
This commit is contained in:
parent
d5e32c843f
commit
851aeae7c7
|
@ -152,6 +152,18 @@ class GroupServerStore(SQLBaseStore):
|
||||||
an order of 1 will put the room first. Otherwise, the room gets
|
an order of 1 will put the room first. Otherwise, the room gets
|
||||||
added to the end.
|
added to the end.
|
||||||
"""
|
"""
|
||||||
|
room_in_group = self._simple_select_one_onecol_txn(
|
||||||
|
txn,
|
||||||
|
table="group_rooms",
|
||||||
|
keyvalues={
|
||||||
|
"group_id": group_id,
|
||||||
|
"room_id": room_id,
|
||||||
|
},
|
||||||
|
retcol="room_id",
|
||||||
|
allow_none=True,
|
||||||
|
)
|
||||||
|
if not room_in_group:
|
||||||
|
raise SynapseError(400, "room not in group")
|
||||||
|
|
||||||
if category_id is None:
|
if category_id is None:
|
||||||
category_id = _DEFAULT_CATEGORY_ID
|
category_id = _DEFAULT_CATEGORY_ID
|
||||||
|
@ -426,6 +438,19 @@ class GroupServerStore(SQLBaseStore):
|
||||||
an order of 1 will put the user first. Otherwise, the user gets
|
an order of 1 will put the user first. Otherwise, the user gets
|
||||||
added to the end.
|
added to the end.
|
||||||
"""
|
"""
|
||||||
|
user_in_group = self._simple_select_one_onecol_txn(
|
||||||
|
txn,
|
||||||
|
table="group_users",
|
||||||
|
keyvalues={
|
||||||
|
"group_id": group_id,
|
||||||
|
"user_id": user_id,
|
||||||
|
},
|
||||||
|
retcol="user_id",
|
||||||
|
allow_none=True,
|
||||||
|
)
|
||||||
|
if not user_in_group:
|
||||||
|
raise SynapseError(400, "user not in group")
|
||||||
|
|
||||||
if role_id is None:
|
if role_id is None:
|
||||||
role_id = _DEFAULT_ROLE_ID
|
role_id = _DEFAULT_ROLE_ID
|
||||||
else:
|
else:
|
||||||
|
|
Loading…
Reference in New Issue