Blocked roles will no longer trigger an error during unblock

Author: fourjr

CHANGELOG.md

@@ -28,6 +28,7 @@ however, insignificant breaking changes do not guarantee a major version bump, s
 - Level permissions were not checked if command permissions were set.
 - Regex autotriggers were not working if term was in the middle of strings.
 - `?blocked` now no longers show blocks that have expired.
+- Blocked roles will no longer trigger an error during unblock
 
 ### Internal
 

bot.py

@@ -673,18 +673,18 @@ def check_manual_blocked_roles(self, author: discord.Member) -> bool:
                         end_time = re.search(r"%([^%]+?)%", blocked_reason)
                         if end_time is not None:
                             logger.warning(
-                                r"Deprecated time message for user %s, block and unblock again to update.",
-                                author.name,
+                                r"Deprecated time message for role %s, block and unblock again to update.",
+                                r.name,
                             )
 
                     if end_time is not None:
                         after = (datetime.fromisoformat(end_time.group(1)) - now).total_seconds()
                         if after <= 0:
                             # No longer blocked
-                            self.blocked_users.pop(str(author.id))
-                            logger.debug("No longer blocked, user %s.", author.name)
+                            self.blocked_roles.pop(str(r.id))
+                            logger.debug("No longer blocked, role %s.", r.name)
                             return True
-                    logger.debug("User blocked, user %s.", author.name)
+                    logger.debug("User blocked, role %s.", r.name)
                     return False
 
         return True