Added events for variables when they change

Author: DanielEverland

Assets/SO Architecture/Editor/Inspectors/BaseVariableEditor.cs

@@ -48,8 +48,17 @@ public override void OnInspectorGUI()
     }
     protected void DrawValue()
     {
-        string content = "Cannot display value. No PropertyDrawer for (" + Target.Type + ") [" + Target.ToString() + "]";
-        GenericPropertyDrawer.DrawPropertyDrawer(Target.Type, _valueProperty, new GUIContent(content, content));
+        using (var scope = new EditorGUI.ChangeCheckScope())
+        {
+            string content = "Cannot display value. No PropertyDrawer for (" + Target.Type + ") [" + Target.ToString() + "]";
+            GenericPropertyDrawer.DrawPropertyDrawer(Target.Type, _valueProperty, new GUIContent(content, content));
+
+            if (scope.changed)
+            {
+                // Value changed, raise events
+                Target.Raise();
+            }
+        }
     }
     protected void DrawClampedFields()
     {

Assets/SO Architecture/Events/Game Events/GameEventBase.cs

@@ -4,6 +4,7 @@
 public abstract class GameEventBase<T> : GameEventBase, IGameEvent<T>, IStackTraceObject
 {
     private readonly List<IGameEventListener<T>> _typedListeners = new List<IGameEventListener<T>>();
+    private readonly List<System.Action<T>> _typedActions = new List<System.Action<T>>();
 
 #if UNITY_EDITOR
     [SerializeField]
@@ -18,6 +19,12 @@ public void Raise(T value)
 
         for (int i = _listeners.Count - 1; i >= 0; i--)
             _listeners[i].OnEventRaised();
+
+        for (int i = _typedActions.Count - 1; i >= 0; i--)
+            _typedActions[i](value);
+
+        for (int i = _typedActions.Count - 1; i >= 0; i--)
+            _actions[i]();
     }
     public void AddListener(IGameEventListener<T> listener)
     {
@@ -29,6 +36,16 @@ public void RemoveListener(IGameEventListener<T> listener)
         if (_typedListeners.Contains(listener))
             _typedListeners.Remove(listener);
     }
+    public void AddListener(System.Action<T> action)
+    {
+        if (!_typedActions.Contains(action))
+            _typedActions.Add(action);
+    }
+    public void RemoveListener(System.Action<T> action)
+    {
+        if (_typedActions.Contains(action))
+            _typedActions.Remove(action);
+    }
     public override string ToString()
     {
         return "GameEventBase<" + typeof(T) + ">";
@@ -37,6 +54,7 @@ public override string ToString()
 public abstract class GameEventBase : SOArchitectureBaseObject, IGameEvent, IStackTraceObject
 {
     protected readonly List<IGameEventListener> _listeners = new List<IGameEventListener>();
+    protected readonly List<System.Action> _actions = new List<System.Action>();
 
     public List<StackTraceEntry> StackTraces { get { return _stackTraces; } }
     private List<StackTraceEntry> _stackTraces = new List<StackTraceEntry>();
@@ -60,6 +78,9 @@ public void Raise()
 
         for (int i = _listeners.Count - 1; i >= 0; i--)
             _listeners[i].OnEventRaised();
+
+        for (int i = _actions.Count - 1; i >= 0; i--)
+            _actions[i]();
     }
     public void AddListener(IGameEventListener listener)
     {
@@ -71,4 +92,14 @@ public void RemoveListener(IGameEventListener listener)
         if (_listeners.Contains(listener))
             _listeners.Remove(listener);
     }
+    public void AddListener(System.Action action)
+    {
+        if (!_actions.Contains(action))
+            _actions.Add(action);
+    }
+    public void RemoveListener(System.Action action)
+    {
+        if (_actions.Contains(action))
+            _actions.Remove(action);
+    }
 }

Assets/SO Architecture/References/BaseReference.cs

@@ -28,6 +28,26 @@ public TBase Value
                 _constantValue = value;
         }
     }
+    public void AddListener(IGameEventListener listener)
+    {
+        if (_variable != null)
+            _variable.AddListener(listener);
+    }
+    public void RemoveListener(IGameEventListener listener)
+    {
+        if (_variable != null)
+            _variable.RemoveListener(listener);
+    }
+    public void AddListener(System.Action action)
+    {
+        if (_variable != null)
+            _variable.AddListener(action);
+    }
+    public void RemoveListener(System.Action action)
+    {
+        if (_variable != null)
+            _variable.AddListener(action);
+    }
     public override string ToString()
     {
         return Value.ToString();

Assets/SO Architecture/Variables/BaseVariable.cs

@@ -1,17 +1,39 @@
 using UnityEngine;
 
-public abstract class BaseVariable : SOArchitectureBaseObject
+public abstract class BaseVariable : GameEventBase
 {
     public abstract bool ReadOnly { get; }
     public abstract System.Type Type { get; }
     public abstract object BaseValue { get; set; }
 }
 public abstract class BaseVariable<T> : BaseVariable
 {
-    public virtual T Value { get { return _value; } set { _value = SetValue(value); } }
+    public virtual T Value
+    {
+        get
+        {
+            return _value;
+        }
+        set
+        {
+            _value = SetValue(value);
+            Raise();
+        }
+    }
     public override bool ReadOnly { get { return _readOnly; } }    
     public override System.Type Type { get { return typeof(T); } }
-    public override object BaseValue { get { return _value; } set { _value = SetValue((T)value); } }
+    public override object BaseValue
+    {
+        get
+        {
+            return _value;
+        }
+        set
+        {
+            _value = SetValue((T)value);
+            Raise();
+        }
+    }
 
     [SerializeField]
     protected T _value;