Merge pull request #226 from benwoo1110/small-fixing

Fix error when guild id is None.

Author: LordOfPolls

discord_slash/client.py

@@ -535,6 +535,7 @@ def add_slash_command(
         """
         name = name or cmd.__name__
         name = name.lower()
+        guild_ids = guild_ids if guild_ids else []
         if not all(isinstance(item, int) for item in guild_ids):
             raise error.IncorrectGuildIDType(
                 f"The snowflake IDs {guild_ids} given are not a list of integers. Because of discord.py convention, please use integer IDs instead. Furthermore, the command '{name}' will be deactivated and broken until fixed."
@@ -616,7 +617,8 @@ def add_subcommand(
         name = name or cmd.__name__
         name = name.lower()
         description = description or getdoc(cmd)
-        if guild_ids and not all(isinstance(item, int) for item in guild_ids):
+        guild_ids = guild_ids if guild_ids else []
+        if not all(isinstance(item, int) for item in guild_ids):
             raise error.IncorrectGuildIDType(
                 f"The snowflake IDs {guild_ids} given are not a list of integers. Because of discord.py convention, please use integer IDs instead. Furthermore, the command '{name}' will be deactivated and broken until fixed."
             )

discord_slash/cog_ext.py

@@ -58,11 +58,10 @@ def wrapper(cmd):
         else:
             opts = options
 
-        if guild_ids is not None:
-            if not all(isinstance(item, int) for item in guild_ids):
-                raise IncorrectGuildIDType(
-                    f"The snowflake IDs {guild_ids} given are not a list of integers. Because of discord.py convention, please use integer IDs instead. Furthermore, the command '{name or cmd.__name__}' will be deactivated and broken until fixed."
-                )
+        if guild_ids and not all(isinstance(item, int) for item in guild_ids):
+            raise IncorrectGuildIDType(
+                f"The snowflake IDs {guild_ids} given are not a list of integers. Because of discord.py convention, please use integer IDs instead. Furthermore, the command '{name or cmd.__name__}' will be deactivated and broken until fixed."
+            )
 
         _cmd = {
             "func": cmd,
@@ -147,11 +146,10 @@ def wrapper(cmd):
         else:
             opts = options
 
-        if guild_ids is not None:
-            if not all(isinstance(item, int) for item in guild_ids):
-                raise IncorrectGuildIDType(
-                    f"The snowflake IDs {guild_ids} given are not a list of integers. Because of discord.py convention, please use integer IDs instead. Furthermore, the command '{name or cmd.__name__}' will be deactivated and broken until fixed."
-                )
+        if guild_ids and not all(isinstance(item, int) for item in guild_ids):
+            raise IncorrectGuildIDType(
+                f"The snowflake IDs {guild_ids} given are not a list of integers. Because of discord.py convention, please use integer IDs instead. Furthermore, the command '{name or cmd.__name__}' will be deactivated and broken until fixed."
+            )
 
         _cmd = {
             "func": None,