|  | Commit message (Collapse) | Author | Age | Lines | 
|---|
| ... |  | 
| | | | | | | | | | | |  | 
| | | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | 
| | | | | | | | | | | It's unnecessarily precise to do an fetch user API call in order to
distinguish between a user and a reason. Furthermore, a User object
isn't actually required for infraction searches - only an ID is. | 
| | | | | | | | | |\ \ |  | 
| | | | | | | | | | | | |  | 
| | | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | 
| | | | | | | | | | | | The point of format_user is to have a consistent format across the code
base. That should apply to tests too. | 
| | | | | | | | | | | | |  | 
| | | | | | | | | |\ \ \ |  | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Using our expanded API is more efficient than making a request to the
Discord API for potentially every user in the search results. The data
may not be up to date, but that's an acceptable compromise. | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | A lot of redundant code existed between the message and eval filters. | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | The API was change a long time ago to not allow such a situation. | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | `format_user` isn't used in the apply mod log cause it already shows
both the old and new nicknames elsewhere. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | This will be used a lot when sending mod logs and will help with
reducing redundancy and maintaining consistency. | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | No longer being used anywhere. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Mocking the bot commands channel constant no longer worked after
switching to `in_whitelist_check`. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | They should only be shown in mod channels, not all staff channels.
Fix #1127 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Fix #1125 | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | `in_whitelist_check` is a convenient utility that does the same thing as
the previous code. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | the modified user data. | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | instead of creating and updating a single user at a time, a list of dicts will be sent for bulk update and creation. | 
| |\| | | | | | | | | | | |  | 
| | |\ \ \ \ \ \ \ \ \ \ \  
| | |_|_|_|_|_|/ / / / /  
| |/| | | | | | | | | | | Move cogs to subdirectories | 
| | | |\ \ \ \ \ \ \ \ \ \  
| | |/ / / / / / / / / /  
| |/| | | | | | | | | | |  | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Being explicit is nice, but the list of extensions to load has gotten
quite long. It's a bit of an eyesore.
It's still fairly easy to temporarily exclude extensions: just remove
them from the set. Granted, being able to comment them out was more
convenient. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Makes the cog cleaner and makes the functions more accessible for other
modules. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Yes, that's a real word. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | PEP 8 recommends absolute imports over relative ones. | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | The directory contains modules, which are extensions. It only indirectly
contains cogs through the extensions. Therefore, a technically more
accurate name is "extensions", or "exts" when abbreviated. Furthermore,
"exts" is consistent with SeasonalBot. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Since `pkgutil.walk_packages` imports packages it comes across, it's
best to avoid potential side effects from imports. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | It's convenient for users to type less to specify the exception they
want. Only require a qualified name if an unqualified name is ambiguous
(i.e. two modules in different subpackages have identical names). | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Discover extensions recursively and ignore any modules/packages whose
names start with an underscore. | 
| | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | Bunch of modules still rely on importing the cog directly from the
moderation package. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | This naming scheme will make them easy to distinguish from extensions. | 
| | | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | 
| | | | | | | | | | | | | The cog tests structure should mirror the structure of the cogs folder.
Fix some import/patch paths which broke due to the restructure. | 
| | | | | | | | | | | | | |  | 
| |/ / / / / / / / / / /  
| | | | | | | | | | |   
| | | | | | | | | | |   
| | | | | | | | | | | | Added method to recursively GET users if paginated and another method to parse URL and return endpoint and query parameters. |