fix: 🐛 Add permission check to Role.is_assignable() (#3048)

Co-authored-by: Nelo <41271523+NeloBlivion@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: 3 people

CHANGELOG.md

@@ -14,6 +14,9 @@ These changes are available on the `master` branch, but have not yet been releas
 
 ### Changed
 
+- Updated `Role.is_assignable()` to also check whether the bot has the `MANAGE_ROLES`
+  permission. ([#3048](https://github.com/Pycord-Development/pycord/pull/3048))
+
 ### Fixed
 
 - Fixed `RawMessageUpdateEvent.cached_message` being always `None` even when the message

discord/role.py

@@ -443,15 +443,26 @@ def is_integration(self) -> bool:
         return self.tags is not None and self.tags.is_integration()
 
     def is_assignable(self) -> bool:
-        """Whether the role is able to be assigned or removed by the bot.
+        """Whether the role is able to be assigned or removed by the bot. This checks whether all of the following conditions are true:
+
+        - The role is not the guild's :attr:`Guild.default_role`
+
+        - The role is not managed
+
+        - The bot has the :attr:`~Permissions.manage_roles` permission
+
+        - The bot's top role is above this role
 
         .. versionadded:: 2.0
+        .. versionchanged:: 2.7.1
+            Added check for :attr:`~Permissions.manage_roles` permission
         """
         me = self.guild.me
         return (
             not self.is_default()
             and not self.managed
-            and (me.top_role > self or me.id == self.guild.owner_id)
+            and me.guild_permissions.manage_roles
+            and me.top_role > self
         )
 
     def is_available_for_purchase(self) -> bool: