Fix member.ban and TypeError in guild.ban (#1666)

* Adjust defaults for `delete_message_` * Update Member.ban to support delete_message_seconds * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update guild.py * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update default * Fix CI * update defaults Co-authored-by: BobDotCom <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Pycord-Development · Oct 4, 2022 · d5c59ea · d5c59ea
1 parent 30e0de4
commit d5c59ea
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/discord/guild.py b/discord/guild.py
@@ -3071,7 +3071,9 @@ async def ban(
                 "delete_message_seconds and delete_message_days are mutually exclusive."
             )
 
-        if not (0 <= delete_message_seconds <= 604800):
+        if delete_message_seconds is not None and not (
+            0 <= delete_message_seconds <= 604800
+        ):
             raise TypeError(
                 "delete_message_seconds must be between 0 and 604800 seconds."
             )

diff --git a/discord/member.py b/discord/member.py
@@ -663,15 +663,19 @@ def timed_out(self) -> bool:
     async def ban(
         self,
         *,
-        delete_message_days: Literal[0, 1, 2, 3, 4, 5, 6, 7] = 1,
+        delete_message_seconds: int | None = None,
+        delete_message_days: Literal[0, 1, 2, 3, 4, 5, 6, 7] | None = None,
         reason: str | None = None,
     ) -> None:
         """|coro|
 
         Bans this member. Equivalent to :meth:`Guild.ban`.
         """
         await self.guild.ban(
-            self, reason=reason, delete_message_days=delete_message_days
+            self,
+            reason=reason,
+            delete_message_seconds=delete_message_seconds,
+            delete_message_days=delete_message_days,
         )
 
     async def unban(self, *, reason: str | None = None) -> None: