Swapped Selector to RX

Author: MystikalPooka

Assets/Scripts/AI/BehaviorManager.cs

@@ -81,7 +81,7 @@ IEnumerator Start()
             {
                 yield return Runner.Tick()
                                    .ToObservable(true)
-                                   .Subscribe(xr => Debug.Log("Starting Tick on Runner"), e => Debug.LogError("Error: " + e)).AddTo(this);
+                                   .Subscribe(xr => { }, e => Debug.LogError("Error: " + e)).AddTo(this);
                 yield return new WaitForSeconds(SecondsBetweenTicks);
                 if (TimesToTick > 1) --TimesToTick;
             }

Assets/Scripts/AI/Components/Selector.cs

@@ -26,23 +26,17 @@ public override IEnumerator Tick(WaitForSeconds delayStart = null)
             CurrentState = (BehaviorState.Running);
             foreach (BehaviorTreeElement behavior in Children)
             {
-                yield return behavior.Tick().ToObservable().Subscribe(_ =>
-                {
-
-                });
+                if (CurrentState != BehaviorState.Running) yield break;
 
-                if (behavior.CurrentState != BehaviorState.Fail)
+                yield return behavior.Tick().ToObservable().Subscribe(_ =>
                 {
-                    this.CurrentState = behavior.CurrentState;
-
-                    if (this.CurrentState == BehaviorState.Success)
+                    if (behavior.CurrentState == BehaviorState.Success)
                     {
-                        UnityEngine.Debug.LogError("Selector is success");
-                        //This selector has completed, break out of the operation
-                        yield break;
+                        this.CurrentState = behavior.CurrentState;
+                        return;
                     }
-                }
-                Debug.LogError("Selector is fail");
+                });
+
             }
             //if it gets here, it went through all subbehaviors and had no successes
             CurrentState = BehaviorState.Fail;