| Commit message (Collapse) | Author | Age | Lines |
| |
|
| |
Co-authored-by: Bluenix <[email protected]>
|
| |\ |
|
| | | |
|
| | |
| |
| | |
Put the literal converter before the Member converter so that "me"/"m" isn't attempted to be converted to a Member.
|
| | |\
| | |
| | | |
Add support for `!infractions by <m|me|uid>`
|
| | | |\
| | |/
| |/| |
|
| | | |
| | |
| | | |
Co-authored-by: Xithrius <[email protected]>
|
| | |\ \
| | | |
| | | | |
Refactor ModLog
|
| | | | | |
|
| | |/ / |
|
| | |\ \
| | | |
| | | | |
Filter the same kind of messages in both message deletion listeners.
|
| | |/ / |
|
| | | |
| | |
| | |
| | | |
(#1937)
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* Don't Exit Token Filtering Early On URLs
The token filtering function would exit early if it detected a URL
within the message, but it made no extra checks to ensure there weren't
other tokens within that message that would trigger it. This made
sense when the filtering logic was written, but it's been modified since
to introduce this bug.
Regression tests included.
Signed-off-by: Hassan Abouelela <[email protected]>
* Links Advisory In Token Filter Tests
Adds a link to the advisory with reasoning for the existence of the
test.
Signed-off-by: Hassan Abouelela <[email protected]>
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* Add sql-fstring tag
* Correct link and wording
* Correction to grammar and wording
Also adds a semicolon
* Add missing "
Co-authored-by: Bluenix <[email protected]>
Co-authored-by: TizzySaurus <[email protected]>
|
| | | |
| | |
| | | |
Migrate to `og_blurple`
|
| | |\ \
| | | |
| | | |
| | | |
| | | | |
python-discord/consider-parent-channels-when-checking-mod-channels
Consider parent channels when checking mod channels
|
| | |/ / |
|
| | |\ \ |
|
| | |/ /
| | |
| | |
| | | |
Move text from footer to description.
|
| | |\ \
| | | |
| | | | |
Clean Cog Rework
|
| | | |\ \
| | |/ /
| |/| | |
|
| | |\ \ \
| | | | |
| | | | | |
Unpin All Messages When Moving Help Channels
|
| | | |\ \ \
| | |/ / /
| |/| | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Occasional hiccups in the Discord API would cause unpinning in help
channel to sometimes fails. This gets around that by unpinning all
messages when making the channel available.
Signed-off-by: Hassan Abouelela <[email protected]>
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Discussion in the pull request raised some legitimate use cases for supplying a time range for multiple channels (e.g clean the last couple of minutes instead of specifying number of messages to traverse).
|
| | | | | | |
|
| | | | |\ \
| | | |/ /
| | |/| | |
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|
| | | | |\ \ |
|
| | | | | | | |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Added a cog_command_error method that sets cleaning to False when a command ends on an exception.
I don't have anything in mind that might cause this, but it will ensure that in any case the cog will still be usable.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
Handle cases where there are no enclosing backticks, and where the regex pattern is invalid.
|
| | | | | | | |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
When providing a user ID it would clash with `traverse` which came first.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
In case of old messages, it would delete the old messages first, and only then bulk delete the remainder, which affected logging.
This commit corrects the deletion order.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- Tells the user if clean cancel was attempted with no ongoing clean.
- Fixes MaxConcurrencyReached call bug. There was a missing argument, and it shouldn't invoke the help embed anyway, so it's now a message.
- Some code refactoring.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
The regex was lowercased, even though regex patterns are case sensitive.
Also adds the DOTALL flag.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
Co-authored-by: Shivansh-007 <[email protected]>
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
After discussion, backticks seems like the preferrable formatting as it also cancels Discord's formatting.
Additionally removed the Optionals from the last args in the commands, to not silently ignore incorrect input.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | | |
Cleaning in the same time range across several channels seems like an arbitrary decision.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
The subcommands are kept simple and with few arguments, as they deal with most cases and their usage shouldn't be cumbersome.
However we might to clean by criteria of several functionalities offered by the subcommands, for example delete a specific user's messages but only those that contain a certain pattern.
For this reason the top-level command can now accept all arguments available in any of the subcommands, and will combine the criteria.
Because the channels list has to be the last argument in order to accept either a list of channel or "*", I had to force a specific pattern in the regex argument for it to not consume the first channel specified.
The regex argument must now have an "r" prefix and be enclosed in single quotes. For example: r'\d+'. For patterns with spaces the whole thing still needs to be enclosed in double quotes.
For consistency the `clean regex` subcommand was changed to require the same.
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
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.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- The two subcommands can now accept a time delta and an ISO date time in addition to messages.
- The two limits are now exclusive. Meaning cleaning until a message will not delete that message.
- Added a separate predicate for the `until` case, as the combination of that command and cache usage would result in incorrect behavior.
Additionally, deleting from cache now correctly traverses only `traverse` messages, rather than trying to delete `traverse` messages.
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This commit changes the clean command to send a message instead of raising BadArgument when no messages are found.
Not finding messages is not an error, and doesn't necessarily require the help embed to spring up, just that the parameters might need tweaking.
|
| | | | | | | |
|