From d807324eef804ec4ea002ef34063046e4fdaeca5 Mon Sep 17 00:00:00 2001
From: kwzrd <kwzrd@pythondiscord.com>
Date: Tue, 4 Aug 2020 18:40:03 +0200
Subject: Verification: implement `_kick_members` helper

---
 bot/cogs/verification.py | 24 +++++++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)

diff --git a/bot/cogs/verification.py b/bot/cogs/verification.py
index 683e60ddb..94c21a568 100644
--- a/bot/cogs/verification.py
+++ b/bot/cogs/verification.py
@@ -1,4 +1,3 @@
-import asyncio
 import logging
 import typing as t
 from contextlib import suppress
@@ -55,8 +54,27 @@ class Verification(Cog):
         self.bot = bot
 
     async def _kick_members(self, members: t.Set[discord.Member]) -> int:
-        """Kick `members` from the PyDis guild."""
-        ...
+        """
+        Kick `members` from the PyDis guild.
+
+        Note that this is a potentially destructive operation. Returns the amount of successful
+        requests. Failed requests are logged at info level.
+        """
+        log.info(f"Kicking {len(members)} members from the guild (not verified after {KICKED_AFTER} days)")
+        n_kicked, bad_statuses = 0, set()
+
+        for member in members:
+            try:
+                await member.kick(reason=f"User has not verified in {KICKED_AFTER} days")
+            except discord.HTTPException as http_exc:
+                bad_statuses.add(http_exc.status)
+            else:
+                n_kicked += 1
+
+        if bad_statuses:
+            log.info(f"Failed to kick {len(members) - n_kicked} members due to following statuses: {bad_statuses}")
+
+        return n_kicked
 
     async def _give_role(self, members: t.Set[discord.Member], role: discord.Role) -> int:
         """
-- 
cgit v1.2.3