Merge pull request #96 from OpenCommissioning/extensibility

fix: UnityEvent extension

Author: Preliy

Editor/Scripts/Inspector/DrivePosition.cs

@@ -28,6 +28,7 @@ public override VisualElement CreateInspectorGUI()
 
             var groupEvents = new PropertyGroup("Events");
             groupEvents.Add(new PropertyField{bindingPath = "OnActiveChanged"});
+            groupEvents.Add(new PropertyField{bindingPath = "OnValueChanged"});
 
             container.Add(groupControl);
             container.Add(groupStatus);

Editor/Scripts/Inspector/DriveSimple.cs

@@ -33,6 +33,7 @@ public override VisualElement CreateInspectorGUI()
 
             var groupEvents = new PropertyGroup("Events");
             groupEvents.Add(new PropertyField{bindingPath = "OnActiveChanged"});
+            groupEvents.Add(new PropertyField{bindingPath = "OnValueChanged"});
 
             container.Add(groupControl);
             container.Add(groupStatus);

Editor/Scripts/Inspector/DriveSpeed.cs

@@ -28,6 +28,7 @@ public override VisualElement CreateInspectorGUI()
 
             var groupEvents = new PropertyGroup("Events");
             groupEvents.Add(new PropertyField{bindingPath = "OnActiveChanged"});
+            groupEvents.Add(new PropertyField{bindingPath = "OnValueChanged"});
 
             container.Add(groupControl);
             container.Add(groupStatus);

Runtime/Scripts/Components/Drive.cs

@@ -20,6 +20,7 @@ public abstract class Drive : Actor, IDevice, IMetadataAsset, ICustomInspector,
         public IPropertyReadOnly<DriveState> State => _stateObserver.State;
 
         public UnityEvent<bool> OnActiveChanged;
+        public UnityEvent<float> OnValueChanged;
 
         [SerializeField]
         protected Property<bool> _override = new (false);
@@ -32,11 +33,13 @@ public abstract class Drive : Actor, IDevice, IMetadataAsset, ICustomInspector,
         protected void OnEnable()
         {
             _stateObserver.IsActive.Subscribe(OnActiveChanged.Invoke);
+            _value.Subscribe(OnValueChanged.Invoke);
         }
 
         protected void OnDisable()
         {
             _stateObserver.IsActive.Unsubscribe(OnActiveChanged.Invoke);
+            _value.Unsubscribe(OnValueChanged.Invoke);
         }
 
         protected void Start()

Samples/Demo/0.1 Devices.unity

@@ -551,6 +551,9 @@ MonoBehaviour:
   OnActiveChanged:
     m_PersistentCalls:
       m_Calls: []
+  OnValueChanged:
+    m_PersistentCalls:
+      m_Calls: []
   _override:
     _value: 0
   _link:
@@ -2322,6 +2325,9 @@ MonoBehaviour:
   OnActiveChanged:
     m_PersistentCalls:
       m_Calls: []
+  OnValueChanged:
+    m_PersistentCalls:
+      m_Calls: []
   _override:
     _value: 0
   _link:
@@ -5026,6 +5032,9 @@ MonoBehaviour:
   OnActiveChanged:
     m_PersistentCalls:
       m_Calls: []
+  OnValueChanged:
+    m_PersistentCalls:
+      m_Calls: []
   _override:
     _value: 0
   _link: