smtnhacker / OJHacker

1 stars 2 forks source link

Replaced certain functions with discord.py decorators #3

Closed Jian-Wuyou closed 3 years ago

Jian-Wuyou commented 3 years ago

Changes

Replaced references to main.isadmin and main.has_role with the @commands.has_any_role decorator.
Fixed bare exceptions to avoid unintended exception handling (i.e. used except Exception: instead of except:)
Following Basti's formatting changes, added spaces in lists and around operators
Added guild id check for admin commands via @commands.check and cnst.ALLOWED_GUILDS

Jian-Wuyou commented 3 years ago

Was able to invite the bot to a test server and execute admin commands by giving myself a role with the same name as server roles. Tested with !PRINT_ALL and !PRINT_DB, but presumably also works with !HARD_RESET.

Added decorator @commands.check to admin commands to check if guild is in cnst.ALLOWED_GUILDS