Fixed auto responder over remote

Author: davidhozic

docs/source/changelog.rst

@@ -37,6 +37,11 @@ Glossary
 Releases
 ---------------------
 
+|UNRELEASED| v4.0.4
+=====================
+- Fixed automatic responder's not being removable over a remote connection.
+
+
 v4.0.3
 ====================
 - Fixed object editing window saving to an incorrect index (and removing other objects).

src/daf/convert.py

@@ -158,7 +158,18 @@ def import_class(path: str):
         "custom_decoder": lambda data: re.compile(data["pattern"], data.get("flags", 0))
     },
     responder.DMResponder: {
-        "attrs": ["condition", "action", "constraints"]
+        "attrs": attributes.get_all_slots(responder.DMResponder),
+        "attrs_restore": {
+            "client": None,
+            "event_ctrl": None
+        }
+    },
+    responder.GuildResponder: {
+        "attrs": attributes.get_all_slots(responder.GuildResponder),
+        "attrs_restore": {
+            "client": None,
+            "event_ctrl": None
+        }
     }
 }
 

src/daf/misc/instance_track.py

@@ -5,6 +5,8 @@
 from weakref import WeakValueDictionary
 from functools import wraps
 
+from .attributes import get_all_slots
+
 
 __all__ = (
     "get_by_id",
@@ -59,7 +61,7 @@ def track_id(cls):
     """
     @wraps(cls, updated=[])
     class TrackedClass(cls):
-        if hasattr(cls, "__slots__") and cls.__slots__:  # Don't break classes without slots
+        if hasattr(cls, "__slots__") and get_all_slots(cls):  # Don't break classes without slots
             __slots__ = ["_daf_id", "_tracked_allow_remote"]
             if not hasattr(cls, "__weakref__"):
                 __slots__.append('__weakref__')

src/daf/responder/base.py

@@ -31,6 +31,14 @@ class ResponderBase(ABC):
         before performing an action.
         All of the constraints inside the ``constraints`` list need to be fulfilled.
     """
+    __slots__ = (
+        "condition",
+        "constraints",
+        "action",
+        "event_ctrl",
+        "client",
+    )
+
     def __init__(
         self,
         condition: BaseLogic,

src/daf/responder/dmresponder.py

@@ -20,6 +20,7 @@
 @doc_category("Auto responder", path="responder")
 class DMResponder(ResponderBase):
     __doc__ = "DM responder implementation. " + ResponderBase.__doc__
+    __slots__ = tuple()
 
     @typechecked
     def __init__(

src/daf/responder/guildresponder.py

@@ -20,6 +20,7 @@
 @doc_category("Auto responder", path="responder")
 class GuildResponder(ResponderBase):
     __doc__ = "Guild responder implementation. " + ResponderBase.__doc__
+    __slots__ = tuple()
 
     @typechecked
     def __init__(

src/daf_gui/tod_extensions/method_execution.py

@@ -43,7 +43,7 @@
     },
     daf.ACCOUNT.remove_responder: {
         # ACCOUNT.servers
-        "responder_to_remove": lambda old_info: old_info.data["responders"]
+        "resp": lambda old_info: old_info.data["responders"]
     }
 }