Flatten debug_publish_current_action

Author: Flova

Diff for: dynamic_stack_decider/dynamic_stack_decider/dsd.py

@@ -373,21 +373,23 @@ def debug_publish_current_action(self):
         """
         Publishes the name of the currently active action
         """
-        if self.debug_active:
-            # Check if there is something on the stack
-            if len(self.stack) > 0:
-                # Get the top element
-                stack_top = self.stack[-1][1]
-                # Check if it is an action or a sequence element and retrieve the current action
-                if isinstance(stack_top, AbstractActionElement):
-                    current_action = stack_top
-                elif isinstance(stack_top, SequenceElement):
-                    current_action = stack_top.current_action
-                else:
-                    return
-                # Only publish if the action changed
-                if current_action.name != self.debug_active_action_cache:
-                    # Publish the name of the current action
-                    self.debug_current_action_publisher.publish(String(data=current_action.name))
-                    # Cache the current action name
-                    self.debug_active_action_cache = current_action.name
+        # Check if debugging is active and if there is something on the stack
+        if not self.debug_active or len(self.stack) == 0:
+            return
+
+        # Get the top element
+        stack_top = self.stack[-1][1]
+        # Check if it is an action or a sequence element and retrieve the current action
+        if isinstance(stack_top, AbstractActionElement):
+            current_action = stack_top
+        elif isinstance(stack_top, SequenceElement):
+            current_action = stack_top.current_action
+        else:
+            return
+
+        # Only publish if the action changed
+        if current_action.name != self.debug_active_action_cache:
+            # Publish the name of the current action
+            self.debug_current_action_publisher.publish(String(data=current_action.name))
+            # Cache the current action name
+            self.debug_active_action_cache = current_action.name

Diff for: dynamic_stack_decider/dynamic_stack_decider/tree.py

@@ -86,7 +86,6 @@ def __init__(self, name, parent, parameters=None, unset_parameters=None):
         :param parent: the parent element, None for the root element
         :param parameters: A dictionary of parameters
         :param unset_parameters: A dictionary of parameters that must be set later
-        :type parent: DecisionTreeElement
         """
         # Call the constructor of the superclass
         super().__init__(name, parent, parameters, unset_parameters)
@@ -185,7 +184,6 @@ def __init__(self, name, parent, parameters=None, unset_parameters=None):
         Create a new ActionTreeElement
         :param name: the class name of the corresponding AbstractActionElement
         :param parent: the parent element
-        :type parent: DecisionTreeElement
         :param parameters: A dictionary of parameters
         :param unset_parameters: A dictionary of parameters that must be set later
         """