| Commit message (Collapse) | Author | Lines |
|
|
|
Should return 1st arg (or None) if eval cmd in message, otherwise return
full content.
|
|
The previous parsing method was naïve in assuming there would always
be something preceding the code (e.g. the '!eval' command invocation)
delimited by a space. Now it will only split if it's sure the eval
command was used in the edited message.
|
|
Relock
|
|
|
|
A stray `bot` was removed from the `on_message` listener, causing it to raise an exception rather than generate a `Context` object from incoming verification channel messages.
|
|
|
|
- Changed the log for when the big brother cog doesn't load in the `apply_ban()` method doesn't properly load from a trace to an error.
|
|
- Docstrings for `apply_ban()` have been edited to mention that the method also removes a banned user from the watch list.
- Parameter `banned` in `apply_unwatch()` was changed to `send_message` in order to be more general. Boolean logic was swapped to coincide with that change.
- `apply_unwatch()`'s sent message moved to the bottom of the method for clarity. Added `return`s to the method to exit early if no message needs to be sent.
Signed-off-by: Daniel Brown <[email protected]>
|
|
Co-Authored-By: Mark <[email protected]>
|
|
Co-Authored-By: Mark <[email protected]>
|
|
|
|
|
|
|
|
empty and with this message formatting go wrong.
|
|
Fixed `!tag search any` raises `AttributeError`.
Changed default value of `keywords` from `None` to `'any'`.
This will make it search for keyword `'any'` when there is no keyword.
|
|
|
|
|
|
Co-Authored-By: Mark <[email protected]>
|
|
|
|
* Add a constant for the footer text
* Import constants module rather than its classes
|
|
The tags have now been shifted from the database to being static files and hence the get_tags()
method has undergone changes. It now dosen't fetch from the database but looks at the local files
and we need not call it more than once.
|
|
The tasks extensions loop requires an event loop to exist. To work
around this, it's been mocked.
|
|
|
|
|
|
discord.py yields duplicate Command objects for each alias a command
has, so the duplicates need to be removed on our end.
|
|
* Rename `walk_extensions` to `walk_modules` because some extensions
don't consist of a single module
|
|
Have to check the modules are equal to prevent yielding imported cogs.
|
|
|
|
This will help reduce nesting in the actual test.
|
|
|
|
|
|
|
|
Kaizen:
- Cut down on the size of the import line by changing the imports from bot.constants to instead just importing the constants. This will help clarify where certain constants are coming from.
- The periodic checkpoint message will no longer ping `@everyone` or `@Admins` when the bot detects that it is being ran in a debug environment. Message is now a simple confirmation that the periodic ping method successfully ran.
Signed-off-by: Daniel Brown <[email protected]>
|
|
Bugs fixed:
- Previously, the code would check to see if `'expires_at'` was in the kwargs, which after testing I came to find out that it is regardless of the duration of the ban. It has sense been changed to use a `.get()` in order to do a proper comparison.
- Code previously attempted to load from the `"BigBrother"` cog which is the incorrect spelling. Changed it to `"Big Brother"` to correct this.
Logging Added:
- Additional trace logs added to both the `infractions.py` file as well as `bigbrother.py` to assist with future debugging or testing.
Signed-off-by: Daniel Brown <[email protected]>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
For tests, ideally creating instances of cogs should be avoided to avoid
extra code execution. This function was copied over from discord.py
because their function is not a static method, though it still works as
one. It was probably just a design decision on their part to not make it
static.
|