Fix small problem

VedalAI · Jan 7, 2025 · daef5ff · daef5ff
1 parent 1fb51ad
commit daef5ff
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/Unity/Assets/Actions/ActionWindow.cs b/Unity/Assets/Actions/ActionWindow.cs
@@ -132,11 +132,7 @@ public ActionWindow AddAction(INeuroAction action)
 
             if (action.ActionWindow != null)
             {
-                if (action.ActionWindow == this)
-                {
-                    Debug.LogError($"Action {action.Name} has already been added to this ActionWindow.");
-                }
-                else
+                if (action.ActionWindow != this)
                 {
                     Debug.LogError($"Cannot add action {action.Name} to this ActionWindow because it is already included in another ActionWindow.");
                 }

diff --git a/Unity/Assets/Actions/BaseNeuroAction.cs b/Unity/Assets/Actions/BaseNeuroAction.cs
@@ -56,7 +56,7 @@ public virtual WsAction GetWsAction()
         protected abstract ExecutionResult Validate(ActionJData actionData, out object? parsedData);
         protected abstract UniTask ExecuteAsync(object? data);
 
-        public void SetActionWindow(ActionWindow actionWindow)
+        void INeuroAction.SetActionWindow(ActionWindow actionWindow)
         {
             if (ActionWindow != null)
             {

diff --git a/Unity/Assets/Actions/INeuroAction.cs b/Unity/Assets/Actions/INeuroAction.cs
@@ -23,6 +23,9 @@ public interface INeuroAction
 
         WsAction GetWsAction();
 
+        /// <summary>
+        /// Sets the active action window for this action. This will be called by the SDK and should not be used manually!
+        /// </summary>
         void SetActionWindow(ActionWindow actionWindow);
     }
 }