aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRapptz <[email protected]>2015-10-17 04:13:15 -0400
committerRapptz <[email protected]>2015-10-17 04:13:15 -0400
commita9fd4fc4e358109510ce5be7302d7ce279bb3829 (patch)
treeb5d57b4bfc19940241fc50c83cdb967e06a23b31
parentAdd Role.managed property. (diff)
downloaddiscord.py-a9fd4fc4e358109510ce5be7302d7ce279bb3829.tar.xz
discord.py-a9fd4fc4e358109510ce5be7302d7ce279bb3829.zip
Better detection for the @everyone role.
-rw-r--r--discord/role.py9
-rw-r--r--discord/server.py2
2 files changed, 8 insertions, 3 deletions
diff --git a/discord/role.py b/discord/role.py
index ce0b510a..2fecfdf9 100644
--- a/discord/role.py
+++ b/discord/role.py
@@ -197,7 +197,8 @@ class Role(object):
A boolean representing if the role will be displayed separately from other members.
.. attribute:: position
- The position of the role.
+ The position of the role. This number is usually positive. A non-positive value indicates that
+ this is the `@everyone` role.
.. attribute:: managed
A boolean indicating if the role is managed by the server through some form of integration
@@ -211,8 +212,12 @@ class Role(object):
self.id = kwargs.get('id')
self.name = kwargs.get('name')
self.permissions = Permissions(kwargs.get('permissions', 0))
- self.position = kwargs.get('position', -1)
+ self.position = kwargs.get('position', 0)
self.colour = Colour(kwargs.get('color', 0))
self.hoist = kwargs.get('hoist', False)
self.managed = kwargs.get('managed', False)
self.color = self.colour
+
+ def is_everyone(self):
+ """Checks if the role is the @everyone role."""
+ return self.position == -1
diff --git a/discord/server.py b/discord/server.py
index a50f0898..1df6fd9c 100644
--- a/discord/server.py
+++ b/discord/server.py
@@ -151,5 +151,5 @@ class Server(object):
def get_default_role(self):
"""Gets the @everyone role that all members have by default."""
for role in self.roles:
- if role.name == '@everyone':
+ if role.is_everyone():
return role