aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar mbaruh <[email protected]>2021-08-29 13:47:37 +0300
committerGravatar mbaruh <[email protected]>2021-08-29 13:47:37 +0300
commit9f124b9eefd24bd1e3bc7210361fe927e8f9eeba (patch)
tree3f91b2a2ca32d1faa72155053efa33b6a10a04f1
parent`until` and `between` overhaul (diff)
Restrict until and between to a single channel
The subcommands should stay simple and answer the most common use cases. Deleting all messages within a time range across many channels seems esoteric and gives just more room for mistakes.
-rw-r--r--bot/exts/moderation/clean.py19
1 files changed, 7 insertions, 12 deletions
diff --git a/bot/exts/moderation/clean.py b/bot/exts/moderation/clean.py
index 90f7f3e03..6c7f3c22d 100644
--- a/bot/exts/moderation/clean.py
+++ b/bot/exts/moderation/clean.py
@@ -432,21 +432,19 @@ class Clean(Cog):
self,
ctx: Context,
until: CleanLimit,
- use_cache: Optional[bool] = True,
- *,
- channels: Optional[CleanChannels] = None) -> None:
+ channel: Optional[TextChannel] = None
+ ) -> None:
"""
Delete all messages until a certain limit.
A limit can be either a message, and ISO date-time string, or a time delta.
- If a message is specified, `channels` cannot be specified.
+ If a message is specified, `channel` cannot be specified.
"""
await self._clean_messages(
CleanMessages.message_limit,
ctx,
- channels=channels,
+ channels=[channel] if channel else None,
first_limit=until,
- use_cache=use_cache
)
@clean_group.command(name="between", aliases=["after-until", "from-to"])
@@ -456,9 +454,7 @@ class Clean(Cog):
ctx: Context,
first_limit: CleanLimit,
second_limit: CleanLimit,
- use_cache: Optional[bool] = True,
- *,
- channels: Optional[CleanChannels] = None
+ channel: Optional[TextChannel] = None
) -> None:
"""
Delete all messages within range.
@@ -467,15 +463,14 @@ class Clean(Cog):
A limit can be either a message, and ISO date-time string, or a time delta.
If two messages are specified, they both must be in the same channel.
- If a message is specified, `channels` cannot be specified.
+ If a message is specified, `channel` cannot be specified.
"""
await self._clean_messages(
CleanMessages.message_limit,
ctx,
- channels=channels,
+ channels=[channel] if channel else None,
first_limit=first_limit,
second_limit=second_limit,
- use_cache=use_cache
)
@clean_group.command(name="stop", aliases=["cancel", "abort"])