aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar wookie184 <[email protected]>2022-09-11 11:57:16 +0100
committerGravatar wookie184 <[email protected]>2022-09-11 11:57:16 +0100
commit7e6e3ca11d6a0ac63b739643c056a797f0523bf6 (patch)
treec5bac9d2aa185a90a56e0429270cf722748ae28c
parentOnly allow command in bot-commands, and change some formatting (diff)
Use Member.get_role instead of adding has_role_id util
-rw-r--r--bot/exts/info/patreon.py3
-rw-r--r--bot/utils/members.py5
2 files changed, 1 insertions, 7 deletions
diff --git a/bot/exts/info/patreon.py b/bot/exts/info/patreon.py
index 09a020efc..c4c555f8d 100644
--- a/bot/exts/info/patreon.py
+++ b/bot/exts/info/patreon.py
@@ -10,7 +10,6 @@ from bot.constants import Channels, Guild, Roles, STAFF_PARTNERS_COMMUNITY_ROLES
from bot.decorators import in_whitelist
from bot.log import get_logger
from bot.utils.channel import get_or_fetch_channel
-from bot.utils.members import has_role_id
log = get_logger(__name__)
@@ -38,7 +37,7 @@ def get_patreon_tier(member: discord.Member) -> int:
A patreon tier of 0 indicates the user is not a patreon.
"""
for tier, role_id in PATREON_TIERS:
- if has_role_id(member, role_id):
+ if member.get_role(role_id):
return tier
return 0
diff --git a/bot/utils/members.py b/bot/utils/members.py
index 410c85452..693286045 100644
--- a/bot/utils/members.py
+++ b/bot/utils/members.py
@@ -46,8 +46,3 @@ async def handle_role_change(
)
except discord.HTTPException as e:
log.error(f"Failed to change role for {member} ({member.id}): {e.status} {e.code}")
-
-
-def has_role_id(member: discord.Member, role_id: int) -> bool:
- """Return boolean representing whether a member has the given role."""
- return any(role_id == role.id for role in member.roles)