fix PresetsList saving to file

PepperDashEssentials/Room/Types/EssentialsTechRoom.cs

@@ -28,6 +28,33 @@ namespace PepperDash.Essentials
         private Dictionary<string, string> _currentPresets;
         private ScheduledEventGroup _roomScheduledEventGroup;
 
+        /// <summary>
+        /// 
+        /// </summary>
+        protected override Func<bool> IsWarmingFeedbackFunc
+        {
+            get
+            {
+                return () =>
+                {
+                    return _displays.All(kv => kv.Value.IsWarmingUpFeedback.BoolValue);
+                };
+            }
+        }
+        /// <summary>
+        /// 
+        /// </summary>
+        protected override Func<bool> IsCoolingFeedbackFunc
+        {
+            get
+            {
+                return () =>
+                {
+                    return _displays.All(kv => kv.Value.IsCoolingDownFeedback.BoolValue);
+                };
+            }
+        }
+
         public EssentialsTechRoom(DeviceConfig config) : base(config)
         {
             _config = config.Properties.ToObject<EssentialsTechRoomConfig>();
@@ -111,7 +138,12 @@ namespace PepperDash.Essentials
             foreach (var display in _displays)
             {
                 display.Value.PowerIsOnFeedback.OutputChange +=
-                    (sender, args) => RoomPowerIsOnFeedback.InvokeFireUpdate();
+                    (sender, args) =>
+                    {
+                        RoomPowerIsOnFeedback.InvokeFireUpdate();
+                        IsWarmingUpFeedback.InvokeFireUpdate();
+                        IsCoolingDownFeedback.InvokeFireUpdate();
+                    };
             }
         }
 
@@ -281,16 +313,6 @@ namespace PepperDash.Essentials
 
         #region Overrides of EssentialsRoomBase
 
-        protected override Func<bool> IsWarmingFeedbackFunc
-        {
-            get { return () => false; }
-        }
-
-        protected override Func<bool> IsCoolingFeedbackFunc
-        {
-            get { return () => false; }
-        }
-
         protected override Func<bool> OnFeedbackFunc
         {
             get { return () => RoomPowerIsOn; }
@@ -358,6 +380,8 @@ namespace PepperDash.Essentials
                         feedback.Value.FireUpdate();
                     }
                 };
+
+                return;
             }
 
             i = 0;
@@ -366,6 +390,8 @@ namespace PepperDash.Essentials
                 var tuner = setTopBox;
 
                 trilist.SetStringSigAction(joinMap.CurrentPreset.JoinNumber + i, s => _tunerPresets.Dial(s, tuner.Value));
+
+                i++;
             }
         }
 

essentials-framework/Essentials Core/PepperDashEssentialsBase/Bridges/BridgeBase.cs

@@ -78,7 +78,7 @@ namespace PepperDash.Essentials.Core.Bridges
     /// <summary>
     /// Bridge API using EISC
     /// </summary>
-    public class EiscApiAdvanced : BridgeApi
+    public class EiscApiAdvanced : BridgeApi, ICommunicationMonitor
     {
         public EiscApiPropertiesConfig PropertiesConfig { get; private set; }
 
@@ -89,6 +89,7 @@ namespace PepperDash.Essentials.Core.Bridges
         public EiscApiAdvanced(DeviceConfig dc, BasicTriList eisc) :
             base(dc.Key)
         {
+            CommunicationMonitor = new CrestronGenericBaseCommunicationMonitor(this, Eisc, 120000, 300000);
             JoinMaps = new Dictionary<string, JoinMapBaseAdvanced>();
 
             PropertiesConfig = dc.Properties.ToObject<EiscApiPropertiesConfig>();
@@ -100,6 +101,19 @@ namespace PepperDash.Essentials.Core.Bridges
 
             AddPostActivationAction(LinkDevices);
             AddPostActivationAction(LinkRooms);
+            AddPostActivationAction(RegisterEisc);
+        }
+
+        public override bool CustomActivate()
+        {
+            CommunicationMonitor.Start();
+            return base.CustomActivate();
+        }
+
+        public override bool Deactivate()
+        {
+            CommunicationMonitor.Stop();
+            return base.Deactivate();
         }
 
         private void LinkDevices()
@@ -137,8 +151,6 @@ namespace PepperDash.Essentials.Core.Bridges
                     bridge.LinkToApi(Eisc, d.JoinStart, d.JoinMapKey, this);
                 }
             }
-
-            RegisterEisc();
         }
 
         private void RegisterEisc()
@@ -182,8 +194,6 @@ namespace PepperDash.Essentials.Core.Bridges
 
                 rm.LinkToApi(Eisc, room.JoinStart, room.JoinMapKey, this);
             }
-
-            RegisterEisc();
         }
 
         /// <summary>
@@ -324,6 +334,12 @@ namespace PepperDash.Essentials.Core.Bridges
                 Debug.Console(2, this, "Error in Eisc_SigChange handler: {0}", e);
             }
         }
+
+        #region Implementation of ICommunicationMonitor
+
+        public StatusMonitorBase CommunicationMonitor { get; private set; }
+
+        #endregion
     }
 
     public class EiscApiPropertiesConfig

essentials-framework/Essentials Core/PepperDashEssentialsBase/PepperDash_Essentials_Core.nuspec

@@ -14,7 +14,7 @@
     <tags>crestron 3series 4series</tags>
     <repository type="git" url="https://github.com/PepperDash/Essentials"/>
     <dependencies>
-      <dependency id="PepperDashCore" version="[1.0.43, 1.1.0)"/>
+      <dependency id="PepperDashCore" version="[1.0.44, 1.1.0)"/>
     </dependencies>
   </metadata>
   <files>

essentials-framework/Essentials Core/PepperDashEssentialsBase/Presets/DevicePresets.cs

@@ -266,14 +266,13 @@ namespace PepperDash.Essentials.Core.Presets
             try
             {
                 _fileOps.Enter();
-                var json = JsonConvert.SerializeObject(PresetsList);
+                var pl = new PresetsList {Channels = PresetsList, Name = Name};
+                var json = JsonConvert.SerializeObject(pl, Formatting.Indented);
 
                 using (var file = File.Open(_filePath, FileMode.Truncate))
                 {
                     file.Write(json, Encoding.UTF8);
                 }
-
-                
             }
             finally
             {

packages.config

@@ -1,3 +1,3 @@
 <packages>
-  <package id="PepperDashCore" version="1.0.43" targetFramework="net35" allowedVersions="[1.0,1.1)"/>
+  <package id="PepperDashCore" version="1.0.44" targetFramework="net35" allowedVersions="[1.0,1.1)"/>
 </packages>