Merge pull request #232 from artem30801/artem30801-fix-url-buttons-context

Fix URL buttons context processing
Fix Docs

Author: LordOfPolls

discord_slash/client.py

@@ -908,9 +908,9 @@ def add_component_callback(
         :param Coroutine callback: The coroutine to be called when the component is interacted with. Must accept a single argument with the type :class:`.context.ComponentContext`.
         :param messages: If specified, only interactions from the message given will be accepted. Can be a message object to check for, or the message ID or list of previous two. Empty list will mean that no interactions are accepted.
         :type messages: Union[discord.Message, int, list]
-        :param components: If specified, only interactions with ``custom_id``s of given components will be accepted. Defaults to the name of ``callback`` if ``use_callback_name=True``. Can be a custom ID (str) or component dict (actionrow or button) or list of previous two.
+        :param components: If specified, only interactions with ``custom_id`` of given components will be accepted. Defaults to the name of ``callback`` if ``use_callback_name=True``. Can be a custom ID (str) or component dict (actionrow or button) or list of previous two.
         :type components: Union[str, dict, list]
-        :param use_callback_name: Whether the ``custom_id`` defaults to the name of ``callback`` if unspecified. If ``False``, either `messages`` or ``components`` must be specified.
+        :param use_callback_name: Whether the ``custom_id`` defaults to the name of ``callback`` if unspecified. If ``False``, either ``messages`` or ``components`` must be specified.
         :type use_callback_name: bool
         :param component_type: The type of the component to avoid collisions with other component types. See :class:`.model.ComponentType`.
         :type component_type: Optional[int]
@@ -961,14 +961,14 @@ def extend_component_callback(
         custom_id: str = None,
     ):
         """
-        Registers existing callback object (:class:`.model.ComponentType`)
+        Registers existing callback object (:class:`.model.ComponentCallbackObject`)
         for specific combination of message_id, custom_id, component_type.
 
         :param callback_obj: callback object.
         :type callback_obj: model.ComponentCallbackObject
         :param message_id: If specified, only removes the callback for the specific message ID.
         :type message_id: Optional[.model]
-        :param custom_id: The `custom_id` of the component.
+        :param custom_id: The ``custom_id`` of the component.
         :type custom_id: Optional[str]
         :raises: .error.DuplicateCustomID, .error.IncorrectFormat
         """
@@ -988,7 +988,7 @@ def get_component_callback(
 
         :param message_id: If specified, only removes the callback for the specific message ID.
         :type message_id: Optional[.model]
-        :param custom_id: The `custom_id` of the component.
+        :param custom_id: The ``custom_id`` of the component.
         :type custom_id: Optional[str]
         :param component_type: The type of the component. See :class:`.model.ComponentType`.
         :type component_type: Optional[int]
@@ -1014,7 +1014,7 @@ def remove_component_callback(
 
         :param message_id: If specified, only removes the callback for the specific message ID.
         :type message_id: Optional[int]
-        :param custom_id: The `custom_id` of the component.
+        :param custom_id: The ``custom_id`` of the component.
         :type custom_id: Optional[str]
         :param component_type: The type of the component. See :class:`.model.ComponentType`.
         :type component_type: Optional[int]
@@ -1067,9 +1067,9 @@ def component_callback(
 
         :param messages: If specified, only interactions from the message given will be accepted. Can be a message object to check for, or the message ID or list of previous two. Empty list will mean that no interactions are accepted.
         :type messages: Union[discord.Message, int, list]
-        :param components: If specified, only interactions with ``custom_id``s of given components will be accepted. Defaults to the name of ``callback`` if ``use_callback_name=True``. Can be a custom ID (str) or component dict (actionrow or button) or list of previous two.
+        :param components: If specified, only interactions with ``custom_id`` of given components will be accepted. Defaults to the name of ``callback`` if ``use_callback_name=True``. Can be a custom ID (str) or component dict (actionrow or button) or list of previous two.
         :type components: Union[str, dict, list]
-        :param use_callback_name: Whether the ``custom_id`` defaults to the name of ``callback`` if unspecified. If ``False``, either `messages`` or ``components`` must be specified.
+        :param use_callback_name: Whether the ``custom_id`` defaults to the name of ``callback`` if unspecified. If ``False``, either ``messages`` or ``components`` must be specified.
         :type use_callback_name: bool
         :param component_type: The type of the component to avoid collisions with other component types. See :class:`.model.ComponentType`.
         :type component_type: Optional[int]

discord_slash/cog_ext.py

@@ -190,9 +190,9 @@ def cog_component(
 
     :param messages: If specified, only interactions from the message given will be accepted. Can be a message object to check for, or the message ID or list of previous two. Empty list will mean that no interactions are accepted.
     :type messages: Union[discord.Message, int, list]
-    :param components: If specified, only interactions with ``custom_id``s of given components will be accepted. Defaults to the name of ``callback`` if ``use_callback_name=True``. Can be a custom ID (str) or component dict (actionrow or button) or list of previous two.
+    :param components: If specified, only interactions with ``custom_id`` of given components will be accepted. Defaults to the name of ``callback`` if ``use_callback_name=True``. Can be a custom ID (str) or component dict (actionrow or button) or list of previous two.
     :type components: Union[str, dict, list]
-    :param use_callback_name: Whether the ``custom_id`` defaults to the name of ``callback`` if unspecified. If ``False``, either `messages`` or ``components`` must be specified.
+    :param use_callback_name: Whether the ``custom_id`` defaults to the name of ``callback`` if unspecified. If ``False``, either ``messages`` or ``components`` must be specified.
     :type use_callback_name: bool
     :param component_type: The type of the component to avoid collisions with other component types. See :class:`.model.ComponentType`.
     :type component_type: Optional[int]

discord_slash/dpy_overrides.py

@@ -23,7 +23,7 @@ def get_component(self, custom_id: int) -> typing.Optional[dict]:
         """
         for row in self.components:
             for component in row["components"]:
-                if component["custom_id"] == custom_id:
+                if "custom_id" in component and component["custom_id"] == custom_id:
                     return component