Updated GenericDigitalInputDevice

Compatibility Update to GenericRelayDevice

#205

essentials-framework/Essentials Core/PepperDashEssentialsBase/Crestron IO/Inputs/GenericDigitalInputDevice.cs

@@ -26,23 +26,78 @@ namespace PepperDash.Essentials.Core.CrestronIO
                 return () => InputPort.State;
             }
         }
-
+
-        public GenericDigitalInputDevice(string key, DigitalInput inputPort):
+
-            base(key)
+        public GenericDigitalInputDevice(string key, string name, Func<IOPortConfig, DigitalInput> postActivationFunc,
-        {
+            IOPortConfig config)
-            InputStateFeedback = new BoolFeedback(InputStateFeedbackFunc);
+            : base(key, name)
-
+        {
-            InputPort = inputPort;
+
-
+            AddPostActivationAction(() =>
-            InputPort.StateChange += InputPort_StateChange;
+            {
-
+                InputPort = postActivationFunc(config);
-        }
+
-
+                InputPort.Register();
+
+                InputPort.StateChange += InputPort_StateChange;
+
+            });
+        }
+
+        #region Events
+
         void InputPort_StateChange(DigitalInput digitalInput, DigitalInputEventArgs args)
         {
-            InputStateFeedback.FireUpdate();
+            InputStateFeedback.FireUpdate();
-        }
+        }
-
+
+        #endregion
+
+        #region PreActivate
+
+        private static DigitalInput GetDigitalInput(IOPortConfig dc)
+        {
+            IDigitalInputPorts ioPortDevice;
+
+            if (dc.PortDeviceKey.Equals("processor"))
+            {
+                if (!Global.ControlSystem.SupportsDigitalInput)
+                {
+                    Debug.Console(0, "GetDigitalInput: Processor does not support Digital Inputs");
+                    return null;
+                }
+                ioPortDevice = Global.ControlSystem;
+            }
+            else
+            {
+                var ioPortDev = DeviceManager.GetDeviceForKey(dc.PortDeviceKey) as IDigitalInputPorts;
+                if (ioPortDev == null)
+                {
+                    Debug.Console(0, "GetDigitalInput: Device {0} is not a valid device", dc.PortDeviceKey);
+                    return null;
+                }
+                ioPortDevice = ioPortDev;
+            }
+            if (ioPortDevice == null)
+            {
+                Debug.Console(0, "GetDigitalInput: Device '0' is not a valid IRelayPorts Device", dc.PortDeviceKey);
+                return null;
+            }
+
+            if (dc.PortNumber > ioPortDevice.NumberOfDigitalInputPorts)
+            {
+                Debug.Console(0, "GetDigitalInput: Device {0} does not contain a port {1}", dc.PortDeviceKey, dc.PortNumber);
+            }
+
+            return ioPortDevice.DigitalInputPorts[dc.PortNumber];
+
+
+        }
+
+        #endregion
+
+        #region Bridge Linking
+
         public override void LinkToApi(BasicTriList trilist, uint joinStart, string joinMapKey, EiscApiAdvanced bridge)
         {
             var joinMap = new IDigitalInputJoinMap(joinStart);
@@ -65,98 +120,37 @@ namespace PepperDash.Essentials.Core.CrestronIO
             {
                 Debug.Console(1, this, "Unable to link device '{0}'.  Input is null", Key);
                 Debug.Console(1, this, "Error: {0}", e);
-            }
+            }
-        }
+        }
+
+        #endregion
+
+        #region Factory
+
+        public class GenericDigitalInputDeviceFactory : EssentialsDeviceFactory<GenericDigitalInputDevice>
+        {
+            public GenericDigitalInputDeviceFactory()
+            {
+                TypeNames = new List<string>() { "digitalinput" };
+            }
+
+            public override EssentialsDevice BuildDevice(DeviceConfig dc)
+            {
+                Debug.Console(1, "Factory Attempting to create new Generic Relay Device");
+
+                var props = JsonConvert.DeserializeObject<IOPortConfig>(dc.Properties.ToString());
+
+                if (props == null) return null;
+
+                var portDevice = new GenericDigitalInputDevice(dc.Key, dc.Name, GetDigitalInput, props);
+
+                return portDevice;
+            }
+        }
+
+        #endregion
+
     }
 
-    public class GenericDigitalInputDeviceFactory : EssentialsDeviceFactory<GenericDigitalInputDevice>
-    {
-        public GenericDigitalInputDeviceFactory()
-        {
-            TypeNames = new List<string>() { "digitalinput" };
-        }
-
-        public override EssentialsDevice BuildDevice(DeviceConfig dc)
-        {
-            Debug.Console(1, "Factory Attempting to create new Digtal Input Device");
-
-            var props = JsonConvert.DeserializeObject<IOPortConfig>(dc.Properties.ToString());
-
-            IDigitalInputPorts portDevice;
-
-            if (props.PortDeviceKey == "processor")
-                portDevice = Global.ControlSystem as IDigitalInputPorts;
-            else
-                portDevice = DeviceManager.GetDeviceForKey(props.PortDeviceKey) as IDigitalInputPorts;
-
-            if (portDevice == null)
-                Debug.Console(0, "ERROR: Unable to add digital input device with key '{0}'. Port Device does not support digital inputs", dc.Key);
-            else
-            {
-                var cs = (portDevice as CrestronControlSystem);
-                if (cs == null)
-                {
-                    Debug.Console(0, "ERROR: Port device for [{0}] is not control system", props.PortDeviceKey);
-                    return null;
-                }
-
-                if (cs.SupportsVersiport)
-                {
-                    Debug.Console(1, "Attempting to add Digital Input device to Versiport port '{0}'", props.PortNumber);
-
-                    if (props.PortNumber > cs.NumberOfVersiPorts)
-                    {
-                        Debug.Console(0, "WARNING: Cannot add Vesiport {0} on {1}. Out of range",
-                            props.PortNumber, props.PortDeviceKey);
-                        return null;
-                    }
-
-                    Versiport vp = cs.VersiPorts[props.PortNumber];
-
-                    if (!vp.Registered)
-                    {
-                        var regSuccess = vp.Register();
-                        if (regSuccess == eDeviceRegistrationUnRegistrationResponse.Success)
-                        {
-                            Debug.Console(1, "Successfully Created Digital Input Device on Versiport");
-                            return new GenericVersiportDigitalInputDevice(dc.Key, vp, props);
-                        }
-                        else
-                        {
-                            Debug.Console(0, "WARNING: Attempt to register versiport {0} on device with key '{1}' failed: {2}",
-                                props.PortNumber, props.PortDeviceKey, regSuccess);
-                            return null;
-                        }
-                    }
-                }
-                else if (cs.SupportsDigitalInput)
-                {
-                    Debug.Console(1, "Attempting to add Digital Input device to Digital Input port '{0}'", props.PortNumber);
-
-                    if (props.PortNumber > cs.NumberOfDigitalInputPorts)
-                    {
-                        Debug.Console(0, "WARNING: Cannot register DIO port {0} on {1}. Out of range",
-                            props.PortNumber, props.PortDeviceKey);
-                        return null;
-                    }
-
-                    DigitalInput digitalInput = cs.DigitalInputPorts[props.PortNumber];
-
-                    if (!digitalInput.Registered)
-                    {
-                        if (digitalInput.Register() == eDeviceRegistrationUnRegistrationResponse.Success)
-                        {
-                            Debug.Console(1, "Successfully Created Digital Input Device on Digital Input");
-                            return new GenericDigitalInputDevice(dc.Key, digitalInput);
-                        }
-                        else
-                            Debug.Console(0, "WARNING: Attempt to register digital input {0} on device with key '{1}' failed.",
-                                props.PortNumber, props.PortDeviceKey);
-                    }
-                }
-            }
-            return null;
-        }
-    }
 
 }

essentials-framework/Essentials Core/PepperDashEssentialsBase/Crestron IO/Relay/GenericRelayDevice.cs

@@ -19,22 +19,23 @@ namespace PepperDash.Essentials.Core.CrestronIO
     {
         public Relay RelayOutput { get; private set; }
 
-        public BoolFeedback OutputIsOnFeedback { get; private set; }
+        public BoolFeedback OutputIsOnFeedback { get; private set; }
-
+
-        public GenericRelayDevice(string key, Relay relay):
+        //Maintained for compatibility with PepperDash.Essentials.Core.Devices.CrestronProcessor
-            base(key)
+        public GenericRelayDevice(string key, Relay relay) :
-        {
+            base(key)
-            OutputIsOnFeedback = new BoolFeedback(new Func<bool>(() => RelayOutput.State));
+        {
-
+            OutputIsOnFeedback = new BoolFeedback(new Func<bool>(() => RelayOutput.State));
-            RelayOutput = relay;
+
-            RelayOutput.Register();
+            RelayOutput = relay;
-
+            RelayOutput.Register();
-            RelayOutput.StateChange += RelayOutput_StateChange;
+
+            RelayOutput.StateChange += new RelayEventHandler(RelayOutput_StateChange);
         }
 
         public GenericRelayDevice(string key, string name, Func<IOPortConfig, Relay> postActivationFunc,
             IOPortConfig config)
-            : base(key, name )
+            : base(key, name)
         {
 
             AddPostActivationAction(() =>
@@ -48,6 +49,8 @@ namespace PepperDash.Essentials.Core.CrestronIO
             });
         }
 
+        #region PreActivate
+
         private static Relay GetRelay(IOPortConfig dc)
         {
 
@@ -84,31 +87,40 @@ namespace PepperDash.Essentials.Core.CrestronIO
             }
             
             return relayDevice.RelayPorts[dc.PortNumber];
-
         }
 
+        #endregion
+
+        #region Events
+
         void RelayOutput_StateChange(Relay relay, RelayEventArgs args)
         {
-            OutputIsOnFeedback.FireUpdate();
+            OutputIsOnFeedback.FireUpdate();
         }
 
-        public void OpenRelay()
+        #endregion
-        {
+
-            RelayOutput.State = false;
+        #region Methods
-        }
+
-
+        public void OpenRelay()
-        public void CloseRelay()
+        {
-        {
+            RelayOutput.State = false;
-            RelayOutput.State = true;
+        }
-        }
+
-
+        public void CloseRelay()
-        public void ToggleRelayState()
+        {
-        {
+            RelayOutput.State = true;
-            if (RelayOutput.State == true)
+        }
-                OpenRelay();
+
-            else
+        public void ToggleRelayState()
-                CloseRelay();
+        {
+            if (RelayOutput.State == true)
+                OpenRelay();
+            else
+                CloseRelay();
         }
+
+        #endregion
 
         #region ISwitchedOutput Members
 
@@ -122,8 +134,10 @@ namespace PepperDash.Essentials.Core.CrestronIO
             OpenRelay();
         }
 
-        #endregion
+        #endregion
-
+
+        #region Bridge Linking
+
         public override void LinkToApi(BasicTriList trilist, uint joinStart, string joinMapKey, EiscApiAdvanced bridge)
         {
             var joinMap = new GenericRelayControllerJoinMap(joinStart);
@@ -153,9 +167,13 @@ namespace PepperDash.Essentials.Core.CrestronIO
 
             // feedback for relay state
 
-            OutputIsOnFeedback.LinkInputSig(trilist.BooleanInput[joinMap.Relay.JoinNumber]);
+            OutputIsOnFeedback.LinkInputSig(trilist.BooleanInput[joinMap.Relay.JoinNumber]);
         }
 
+        #endregion
+
+        #region Factory
+
         public class GenericRelayDeviceFactory : EssentialsDeviceFactory<GenericRelayDevice>
         {
             public GenericRelayDeviceFactory()
@@ -227,8 +245,11 @@ namespace PepperDash.Essentials.Core.CrestronIO
                 */
 
             }
-        }
+        }
-
+
+        #endregion
+
+
     }