From 657b9f1f5401b3ea6c7f664a6f1b4bb42608660f Mon Sep 17 00:00:00 2001
From: Neil Dorin <ndorin@pepperdash.com>
Date: Thu, 28 Jan 2021 18:08:06 -0700
Subject: [PATCH] Adds summary help and Room ActivityComponentBase

---
 .../Interfaces/Components/RoomComponents.cs   | 50 ++++++++++++-
 .../Room/Components/RoomComponentBase.cs      | 73 +++++++++++++++++++
 2 files changed, 119 insertions(+), 4 deletions(-)

diff --git a/essentials-framework/Essentials Core/PepperDashEssentialsBase/Interfaces/Components/RoomComponents.cs b/essentials-framework/Essentials Core/PepperDashEssentialsBase/Interfaces/Components/RoomComponents.cs
index f0a69467..aa57fc66 100644
--- a/essentials-framework/Essentials Core/PepperDashEssentialsBase/Interfaces/Components/RoomComponents.cs	
+++ b/essentials-framework/Essentials Core/PepperDashEssentialsBase/Interfaces/Components/RoomComponents.cs	
@@ -41,42 +41,81 @@ namespace PepperDash.Essentials.Core.Interfaces.Components
     /// </summary>
     public interface IRoomActivityComponent : IRoomComponent
     {
+        /// <summary>
+        /// Indicates if the component is enabled
+        /// </summary>
         BoolFeedback IsEnabledFeedback { get; }
 
+        /// <summary>
+        /// Set this value to enable or disable the component
+        /// </summary>
         bool Enable { set; }
+        /// <summary>
+        /// Label to be displayed for the activity on the UI
+        /// </summary>
         string Label { get; }
+        /// <summary>
+        /// Icon to be displayed for the activity on the UI
+        /// </summary>
         string Icon { get;  }
+        /// <summary>
+        /// The component group that will be activated when this activty starts
+        /// </summary>
         IRoomBehaviourGroupComponent Component { get; }
+        /// <summary>
+        /// Determines the order the activities will be displayed on the UI
+        /// </summary>
         int Order { get; }
 
 
+        /// <summary>
+        /// Starts the activity
+        /// </summary>
         void StartActivity();
+        /// <summary>
+        /// Ends the activity
+        /// </summary>
         void EndActivity();
     }
 
     /// <summary>
-    /// Describes a room component that can be "used" by a user
+    /// Describes a room component that can be "activated" 
     /// </summary>
     public interface IActivatableComponent : IRoomComponent
     {
+        /// <summary>
+        /// Indicates if the component is activated
+        /// </summary>
         BoolFeedback ActivatedFeedback { get; }
 
+        /// <summary>
+        /// Activates the component
+        /// </summary>
         void Activate();
+        /// <summary>
+        /// Dactivates the component
+        /// </summary>
         void Deactivate();
     }
 
     /// <summary>
-    /// Describes a room behaviour component.  Is able to contain a collection of components that aggregate
+    /// Describes a group of room behaviour component.  Is able to contain a collection of components that aggregate
     /// together to behave as one
     /// </summary>
     public interface IRoomBehaviourGroupComponent
     {
+        /// <summary>
+        /// A collection of components that work together to achieve a common behaviour
+        /// </summary>
         List<IActivatableComponent> Components { get; }
 
         void ActivateComponents();
         void DeactivateComponents();
     }
 
+    /// <summary>
+    /// Describes an individual room behaviour component
+    /// </summary>
     public interface IRoomBehaviourComponent : IActivatableComponent
     {
 
@@ -84,10 +123,13 @@ namespace PepperDash.Essentials.Core.Interfaces.Components
 
 
     /// <summary>
-    /// Describes a room device component
+    /// Describes a room device component behaviour
     /// </summary>
-    public interface IRoomDeviceComponent<T> : IActivatableComponent where T : EssentialsDevice
+    public interface IDeviceBehaviourComponent<T> : IActivatableComponent where T : EssentialsDevice
     {
+        /// <summary>
+        /// The device this component applies behaviour to
+        /// </summary>
         T Device { get; }
     }
 }
\ No newline at end of file
diff --git a/essentials-framework/Essentials Core/PepperDashEssentialsBase/Room/Components/RoomComponentBase.cs b/essentials-framework/Essentials Core/PepperDashEssentialsBase/Room/Components/RoomComponentBase.cs
index 19da5ffb..3dd00efb 100644
--- a/essentials-framework/Essentials Core/PepperDashEssentialsBase/Room/Components/RoomComponentBase.cs	
+++ b/essentials-framework/Essentials Core/PepperDashEssentialsBase/Room/Components/RoomComponentBase.cs	
@@ -35,5 +35,78 @@ namespace PepperDash.Essentials.Core.Room.Components
 
     }
 
+    /// <summary>
+    /// The base class from which Room Activities should be derived
+    /// </summary>
+    public abstract class RoomActivityComponentBase : IRoomActivityComponent
+    {
+
+        #region IRoomActivityComponent Members
+
+        public BoolFeedback IsEnabledFeedback { get; private set; }
+
+        private bool _enable;
+
+        public bool Enable
+        {
+            set 
+            {
+                if (value != _enable)
+                {
+                    _enable = value;
+                    IsEnabledFeedback.FireUpdate();
+                }
+            }
+        }
+
+        public string Label { get; private set; }
+
+        public string Icon { get; private set; }
+
+        public IRoomBehaviourGroupComponent Component { get; private set; }
+
+        public int Order { get; private set; }
+
+        #endregion
+
+        #region IRoomComponent Members
+
+        public IComponentRoom Parent { get; private set; }
+
+        #endregion
+
+        #region IKeyed Members
+
+        private string _componentKey;
+
+        public string Key
+        {
+            get
+            {
+                return Parent.Key + "-" + _componentKey;
+            }
+        }
+
+        #endregion
+
+        public RoomActivityComponentBase(string key, IComponentRoom parent)
+        {
+            _componentKey = key;
+            Parent = parent;
+
+            IsEnabledFeedback = new BoolFeedback(() => _enable);
+        }
+
+        public void StartActivity()
+        {
+            throw new NotImplementedException();
+        }
+
+        public void EndActivity()
+        {
+            throw new NotImplementedException();
+        }
+    }
+
 
 }
\ No newline at end of file