| Commit message (Collapse) | Author | Age | Lines | 
| ...            |  | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
Have to check the modules are equal to prevent yielding imported cogs.
 | 
| | | | | | | |  | 
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
This will help reduce nesting in the actual test.
 | 
| | | | | | | |  | 
 | 
| | | | | | | |  | 
 | 
| | | | | | | |  | 
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
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.
 | 
| | | |_|_|_|/  
| |/| | | |    | 
 | 
| | | | | | |  | 
 | 
| | | | | | |  | 
 | 
| | |/ / / /  
|/| | | |   
| | | | |   
| | | | |    | 
empty and with this message formatting go wrong.
 | 
| |\ \ \ \ \  
| |/ / / /  
|/| | | |    | 
Removed API dependency for Tags
 | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | |  | 
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.
 | 
| | | | | |  | 
 | 
| | |\ \ \ \  
| | | | | | 
| | | | | | 
| | | | | |  | 
tags_overhaul
 | 
| | | | | | |  | 
 | 
| | | | | | | 
| | | | | | 
| | | | | | 
| | | | | | 
| | | | | |  | 
* 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.
 | 
| | |\ \ \ \  
| | | | | | 
| | | | | | 
| | | | | |  | 
tags_overhaul
 | 
| | | |\ \ \ \  
| |_|/ / / /  
|/| | | | |    | 
 | 
| | |/ / / /   | 
 | 
| | | | | |  | 
 | 
| | | | | |  | 
 | 
| | | | | |  | 
 | 
| | |\ \ \ \   | 
 | 
| | | | | | | 
| | | | | | 
| | | | | | 
| | | | | |  | 
"get_tags()" and "_get_tag()" functions need not be async as we are no longer doing any API call but instead reading from local files.
 | 
| | | | | | | 
| | | | | | 
| | | | | | 
| | | | | |  | 
The pathlib module simplifies opening and reading files, hence the os module and the context manager are no longer used.
 | 
| | |\ \ \ \ \  
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
tags_overhaul
 | 
| | | |\ \ \ \ \   | 
 | 
| | |/ / / / / /  
| | | | | | |   
| | | | | | |   
| | | | | | |    | 
caching it after the tags command is used.
 | 
| | | | | | | |  | 
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
of points on github
 | 
| | | | | | | | 
| | | | | | | 
| | | | | | | 
| | | | | | |  | 
access the static tag files rather than sending an API get request. Removed all methods calling the API so the tags cannot be edited, added nor deleted.
 | 
| | | | | | | |  | 
 | 
| | | | | | | |  | 
 | 
| | | | | | | |  | 
 | 
| | | | | |\ \ \  
| |_|_|_|/ / /  
|/| | | | | |    | 
 | 
| | | | | |\ \ \   | 
 | 
| | | | | |\ \ \ \   | 
 | 
| | | | | |\ \ \ \ \   | 
 | 
| | | | | | | | | | |  | 
 | 
| | | | | | | | | | |  | 
 | 
| | | | | | | | | | |  | 
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
- 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]>
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
Co-Authored-By: Mark <[email protected]>
 | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | |  | 
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]>
 |