src/api/v1/members/uuid/ban.rs

@@ -36,7 +36,6 @@ pub async fn post(
     let member = Member::fetch_one_with_member(&app_state, None, member_uuid).await?;
 
     let caller = Member::check_membership(&mut conn, uuid, member.guild_uuid).await?;
-
     caller
         .check_permission(&app_state, Permissions::BanMember)
         .await?;

src/objects/member.rs

@@ -208,6 +208,10 @@ impl Member {
     }
 
     pub async fn delete(self, conn: &mut Conn) -> Result<(), Error> {
+        if self.is_owner {
+            return Err(Error::Forbidden("Can not ban owner".to_string()));
+        }
+
         delete(guild_members::table)
             .filter(guild_members::uuid.eq(self.uuid))
             .execute(conn)
@@ -217,10 +221,6 @@ impl Member {
     }
 
     pub async fn ban(self, conn: &mut Conn, reason: &String) -> Result<(), Error> {
-        if self.is_owner {
-            return Err(Error::Forbidden("Can not ban owner".to_string()));
-        }
-
         use guild_bans::dsl;
         insert_into(guild_bans::table)
             .values((