diff options
| author | Rapptz <[email protected]> | 2015-10-17 04:13:15 -0400 |
|---|---|---|
| committer | Rapptz <[email protected]> | 2015-10-17 04:13:15 -0400 |
| commit | a9fd4fc4e358109510ce5be7302d7ce279bb3829 (patch) | |
| tree | b5d57b4bfc19940241fc50c83cdb967e06a23b31 | |
| parent | Add Role.managed property. (diff) | |
| download | discord.py-a9fd4fc4e358109510ce5be7302d7ce279bb3829.tar.xz discord.py-a9fd4fc4e358109510ce5be7302d7ce279bb3829.zip | |
Better detection for the @everyone role.
| -rw-r--r-- | discord/role.py | 9 | ||||
| -rw-r--r-- | discord/server.py | 2 |
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 |