feat: idiot proofing
This commit is contained in:
parent
c725d13ca8
commit
e074ca89f9
2 changed files with 3 additions and 4 deletions
|
@ -48,10 +48,6 @@ pub async fn delete(
|
||||||
|
|
||||||
let member = Member::fetch_one_with_member(&app_state, &me, member_uuid).await?;
|
let member = Member::fetch_one_with_member(&app_state, &me, member_uuid).await?;
|
||||||
|
|
||||||
if member.is_owner {
|
|
||||||
return Err(Error::Forbidden("Can not kick owner".to_string()));
|
|
||||||
}
|
|
||||||
|
|
||||||
let deleter = Member::check_membership(&mut conn, uuid, member.guild_uuid).await?;
|
let deleter = Member::check_membership(&mut conn, uuid, member.guild_uuid).await?;
|
||||||
|
|
||||||
deleter
|
deleter
|
||||||
|
|
|
@ -188,6 +188,9 @@ impl Member {
|
||||||
}
|
}
|
||||||
|
|
||||||
pub async fn delete(self, conn: &mut Conn) -> Result<(), Error> {
|
pub async fn delete(self, conn: &mut Conn) -> Result<(), Error> {
|
||||||
|
if member.is_owner {
|
||||||
|
return Err(Error::Forbidden("Can not kick owner".to_string()))
|
||||||
|
}
|
||||||
delete(guild_members::table)
|
delete(guild_members::table)
|
||||||
.filter(guild_members::uuid.eq(self.uuid))
|
.filter(guild_members::uuid.eq(self.uuid))
|
||||||
.execute(conn)
|
.execute(conn)
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue