refactor: SetupInput() and SetupOutput() methods to set standard keys for ports that are instantiated

essentials-framework/Essentials DM/Essentials_DM/Chassis/HdPsXxxController.cs

@@ -1,8 +1,6 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using System.Text;
-using Crestron.SimplSharp;
 using Crestron.SimplSharpPro;
 using Crestron.SimplSharpPro.DeviceSupport;
 using Crestron.SimplSharpPro.DM;
@@ -17,12 +15,9 @@ using PepperDash_Essentials_DM.Config;
 namespace PepperDash_Essentials_DM.Chassis
 {
 	[Description("Wrapper class for all HdPsXxx switchers")]
-	public class HdPsXxxController : CrestronGenericBridgeableBaseDevice, IRoutingNumericWithFeedback, IHasFeedback
+	public class HdPsXxxController : CrestronGenericBridgeableBaseDevice, IRoutingNumericWithFeedback, ICec, IHasHdmiInHdcp, IHasFeedback
 	{
-
 		private readonly HdPsXxx _chassis;
-		private readonly HdPs401 _chassis401;
-		private readonly HdPs621 _chassis621;
 
 		public RoutingPortCollection<RoutingInputPort> InputPorts { get; private set; }
 		public RoutingPortCollection<RoutingOutputPort> OutputPorts { get; private set; }
@@ -67,31 +62,25 @@ namespace PepperDash_Essentials_DM.Chassis
 			InputNameFeedbacks = new FeedbackCollection<StringFeedback>();
 			InputHdcpEnableFeedback = new FeedbackCollection<BoolFeedback>();
 			InputNames = new Dictionary<uint, string>();			
-			//InputNames = props.Inputs;
 
 			OutputPorts = new RoutingPortCollection<RoutingOutputPort>();
 			OutputNameFeedbacks = new FeedbackCollection<StringFeedback>();
 			OutputRouteNameFeedback = new FeedbackCollection<StringFeedback>();
 			OutputNames = new Dictionary<uint, string>();
-			//OutputNames = props.Outputs;
 
 			VideoInputSyncFeedbacks = new FeedbackCollection<BoolFeedback>();
 			VideoOutputRouteFeedbacks = new FeedbackCollection<IntFeedback>();
 
 			if (_chassis.NumberOfOutputs == 1)
-			{
+				AutoRouteFeedback = new BoolFeedback(() => _chassis.PriorityRouteOnFeedback.BoolValue);
-				if (_chassis is HdPs401)
-					_chassis401 = _chassis as HdPs401;
-				if (_chassis is HdPs621)
-					_chassis621 = _chassis as HdPs621;
 
-				AutoRouteFeedback = new BoolFeedback(() => _chassis401.PriorityRouteOnFeedback.BoolValue);
+			InputNames = props.Inputs;
-			}
+			SetupInputs(InputNames);
 
-			SetupInputs(props.Inputs);
+			OutputNames = props.Outputs;
-			SetupOutputs(props.Outputs);
+			SetupOutputs(OutputNames);
 
-			AddPostActivationAction(AddFeedbackCollecitons);
+			//AddPostActivationAction();
 		}
 
 		// input setup
@@ -106,30 +95,46 @@ namespace PepperDash_Essentials_DM.Chassis
 			{
 				Debug.Console(1, this, "props.Input[{0}]: {1}", kvp.Key, kvp.Value);
 			}
-			InputNames = dict;
 
-			for (uint i = 1; i <= _chassis.NumberOfInputs; i++)
+			foreach (var item in _chassis.HdmiInputs)
 			{
-				var index = i;
+				var input = item;
-				var name = string.IsNullOrEmpty(InputNames[index]) ? string.Format("Input {0}", index) : InputNames[index];
+				var index = item.Number;
-				var input = _chassis.Inputs[index];
+				var key = string.Format("hdmiIn{0}", index);
-				var hdmiInput = _chassis.HdmiInputs[index];
+				var name = string.IsNullOrEmpty(InputNames[index]) ? string.Format("HDMI Input {0}", index) : InputNames[index];
-				var dmLiteInput = _chassis.DmLiteInputs[index];
 				
 				InputNameFeedbacks.Add(new StringFeedback(name, () => InputNames[index]));
 
-				// TODO [ ] verify which input type is needed
+				var port = new RoutingInputPort(key, eRoutingSignalType.AudioVideo, eRoutingPortConnectionType.Hdmi, input, this)
+				{
+					FeedbackMatchObject = input
+				};
+				Debug.Console(1, this, "Adding HDMI Input port: {0}", port.Key);
+				InputPorts.Add(port);
+
+				InputHdcpEnableFeedback.Add(new BoolFeedback(name, () => input.InputPort.HdcpSupportOnFeedback.BoolValue));
+
+				VideoInputSyncFeedbacks.Add(new BoolFeedback(name, () => input.VideoDetectedFeedback.BoolValue));				
+			}
+
+			foreach (var item in _chassis.DmLiteInputs)
+			{
+				var input = item;
+				var index = item.Number;
+				var key = string.Format("dmLiteIn{0}", index);
+				var name = string.IsNullOrEmpty(InputNames[index]) ? string.Format("DM Input {0}", index) : InputNames[index];
 				input.Name.StringValue = name;
-				hdmiInput.Name.StringValue = name;
+
-				dmLiteInput.Name.StringValue = name;
+				InputNameFeedbacks.Add(new StringFeedback(name, () => InputNames[index]));
 
 				var port = new RoutingInputPort(name, eRoutingSignalType.AudioVideo, eRoutingPortConnectionType.Hdmi, input, this)
 				{
 					FeedbackMatchObject = input
 				};
+				Debug.Console(0, this, "Adding DM Input port: {0}",port.Key);
 				InputPorts.Add(port);
 				
-				InputHdcpEnableFeedback.Add(new BoolFeedback(name, () => hdmiInput.InputPort.HdcpSupportOnFeedback.BoolValue));
+				InputHdcpEnableFeedback.Add(new BoolFeedback(name, () => input.InputPort.HdcpSupportOnFeedback.BoolValue));
 
 				VideoInputSyncFeedbacks.Add(new BoolFeedback(name, () => input.VideoDetectedFeedback.BoolValue));
 			}
@@ -149,25 +154,23 @@ namespace PepperDash_Essentials_DM.Chassis
 			{
 				Debug.Console(1, this, "props.Output[{0}]: {1}", kvp.Key, kvp.Value);
 			}
-			OutputNames = dict;
 
-			for (uint i = 1; i <= _chassis.NumberOfOutputs; i++)
+			foreach (var item in _chassis.HdmiDmLiteOutputs)
 			{
-				var index = i;
+				var output = item;
+				var index = item.Number;
+				var key = string.Format("hdmiDmLiteOut{0}", index);
 				var name = string.IsNullOrEmpty(OutputNames[index]) ? string.Format("Output {0}", index) : OutputNames[index];
-				var output = _chassis.Outputs[index];
-				var hdmiDmLiteOutput = _chassis.HdmiDmLiteOutputs[index];
-
-				OutputNameFeedbacks.Add(new StringFeedback(name, () => OutputNames[index]));
-
-				// TODO [ ] verify which output type is needed
 				output.Name.StringValue = name;
-				hdmiDmLiteOutput.Name.StringValue = name;
 
-				var port = new RoutingOutputPort(name, eRoutingSignalType.AudioVideo, eRoutingPortConnectionType.Hdmi, output, this)
+				var port = new RoutingOutputPort(key, eRoutingSignalType.AudioVideo, eRoutingPortConnectionType.Hdmi, output, this)
 				{
-					FeedbackMatchObject = output
+					FeedbackMatchObject = output,
+					// set port for CEC
+					Port = output
 				};
+				Debug.Console(0, this, "Adding HdmiDmLite Output port: {0} {1}",
+					port.Key, port.ParentDevice);
 				OutputPorts.Add(port);
 
 				OutputRouteNameFeedback.Add(new StringFeedback(name, () => output.VideoOutFeedback.NameFeedback.StringValue));
@@ -206,10 +209,6 @@ namespace PepperDash_Essentials_DM.Chassis
 
 			_chassis.OnlineStatusChange += _chassis_OnlineStatusChange;
 
-			if (_chassis401 != null) LinkChassis401ToApi(trilist, joinMap);
-
-			if (_chassis621 != null) LinkChassis621ToApi(trilist, joinMap);
-
 			LinkChassisInputsToApi(trilist, joinMap);
 			LinkChassisOutputsToApi(trilist, joinMap);
 
@@ -252,36 +251,17 @@ namespace PepperDash_Essentials_DM.Chassis
 				var indexWithOffset = output - 1;
 
 				trilist.SetUShortSigAction(joinMap.OutputRoute.JoinNumber + indexWithOffset, (a) =>
-					ExecuteNumericSwitch(a, (ushort) output, eRoutingSignalType.AudioVideo));
+					ExecuteNumericSwitch(a, (ushort)output, eRoutingSignalType.AudioVideo));
 
 				OutputNameFeedbacks[outputName].LinkInputSig(trilist.StringInput[joinMap.OutputName.JoinNumber + indexWithOffset]);
 				OutputRouteNameFeedback[outputName].LinkInputSig(trilist.StringInput[joinMap.OutputRoutedName.JoinNumber + indexWithOffset]);
 
 				VideoOutputRouteFeedbacks[outputName].LinkInputSig(trilist.UShortInput[joinMap.OutputRoute.JoinNumber + indexWithOffset]);
 			}
-		}
-
-
-		// links HdPs401 chassis to API
-		private void LinkChassis401ToApi(BasicTriList trilist, HdPsXxxControllerJoinMap joinMap)
-		{
-			trilist.SetSigTrueAction(joinMap.EnableAutoRoute.JoinNumber, () => _chassis401.AutoRouteOn());
-			trilist.SetSigFalseAction(joinMap.EnableAutoRoute.JoinNumber, () => _chassis401.AutoRouteOff());
 
 			AutoRouteFeedback.LinkInputSig(trilist.BooleanInput[joinMap.EnableAutoRoute.JoinNumber]);
 		}
 
-
-		// links HdPs621 chassis to API
-		private void LinkChassis621ToApi(BasicTriList trilist, HdPsXxxControllerJoinMap joinMap)
-		{
-			trilist.SetSigTrueAction(joinMap.EnableAutoRoute.JoinNumber, () => _chassis621.AutoRouteOn());
-			trilist.SetSigFalseAction(joinMap.EnableAutoRoute.JoinNumber, () => _chassis621.AutoRouteOff());
-
-			AutoRouteFeedback.LinkInputSig(trilist.BooleanInput[joinMap.EnableAutoRoute.JoinNumber]);
-		}
-
-
 		#endregion
 	
 
@@ -359,17 +339,9 @@ namespace PepperDash_Essentials_DM.Chassis
 		/// </summary>
 		public void EnableAutoRoute()
 		{
-			if (_chassis.NumberOfInputs != 1) return;
+			if (_chassis.NumberOfInputs == 1) return;
 
-			if (_chassis401 != null)
+			_chassis.AutoRouteOn();
-			{
-				_chassis401.AutoRouteOn();
-			}
-
-			if (_chassis621 != null)
-			{
-				_chassis621.AutoRouteOn();
-			}
 		}
 
 
@@ -378,17 +350,9 @@ namespace PepperDash_Essentials_DM.Chassis
 		/// </summary>
 		public void DisableAutoRoute()
 		{
-			if (_chassis.NumberOfInputs != 1) return;
+			if (_chassis.NumberOfInputs == 1) return;
 
-			if (_chassis401 != null)
+			_chassis.AutoRouteOff();
-			{
-				_chassis401.AutoRouteOff();
-			}
-
-			if (_chassis621 != null)
-			{
-				_chassis621.AutoRouteOff();
-			}
 		}
 
 		#region Events
@@ -417,31 +381,31 @@ namespace PepperDash_Essentials_DM.Chassis
 			switch (eventId)
 			{
 				case DMInputEventIds.VideoDetectedEventId:
-				{
-					Debug.Console(1, this, "Event ID {0}: Updating VideoInputSyncFeedbacks", eventId);
-					foreach (var item in VideoInputSyncFeedbacks)
 					{
-						item.FireUpdate();
+						Debug.Console(1, this, "Event ID {0}: Updating VideoInputSyncFeedbacks", eventId);
+						foreach (var item in VideoInputSyncFeedbacks)
+						{
+							item.FireUpdate();
+						}
+						break;
 					}
-					break;
-				}
 				case DMInputEventIds.InputNameFeedbackEventId:
 				case DMInputEventIds.InputNameEventId:
 				case DMInputEventIds.NameFeedbackEventId:
-				{
+					{
-					Debug.Console(1, this, "Event ID {0}: Updating name feedbacks", eventId);
+						Debug.Console(1, this, "Event ID {0}: Updating name feedbacks", eventId);
 
-					var input = args.Number;
+						var input = args.Number;
-					var name = _chassis.HdmiInputs[input].NameFeedback.StringValue;
+						var name = _chassis.HdmiInputs[input].NameFeedback.StringValue;
 
-					Debug.Console(1, this, "Input {0} Name {1}", input, name);
+						Debug.Console(1, this, "Input {0} Name {1}", input, name);
-					break;
+						break;
-				}
+					}
 				default:
-				{
+					{
-					Debug.Console(1, this, "Uhandled DM Input Event ID {0}", eventId);
+						Debug.Console(1, this, "Uhandled DM Input Event ID {0}", eventId);
-					break;
+						break;
-				}
+					}
 			}
 		}
 
@@ -489,114 +453,6 @@ namespace PepperDash_Essentials_DM.Chassis
 		#endregion		
 
 
-		#region FeedbacksAndFeedbackCollections
-
-
-		/// <summary>
-		/// Add feedback colleciton arrays to feedback collections 
-		/// </summary>
-		/// <param name="feedbackCollections">BoolFeedback[] arrays</param>
-		public void AddCollectionsToList(params FeedbackCollection<BoolFeedback>[] feedbackCollections)
-		{
-			foreach (var item in feedbackCollections.SelectMany(feedbackCollection => feedbackCollections))
-			{
-				AddCollectionsToList(item);
-			}
-		}
-
-
-		/// <summary>
-		/// Add feedback colleciton arrays to feedback collections 
-		/// </summary>
-		/// <param name="feedbackCollections">IntFeedback[] arrays</param>
-		public void AddCollectionsToList(params FeedbackCollection<IntFeedback>[] feedbackCollections)
-		{
-			foreach (var item in feedbackCollections.SelectMany(feedbackCollection => feedbackCollections))
-			{
-				AddCollectionsToList(item);
-			}
-		}
-
-
-		/// <summary>
-		/// Add feedback colleciton arrays to feedback collections 
-		/// </summary>
-		/// <param name="feedbackCollections">StringFeedback[] arrays</param>
-		public void AddCollectionsToList(params FeedbackCollection<StringFeedback>[] feedbackCollections)
-		{
-			foreach (var item in feedbackCollections.SelectMany(feedbackCollection => feedbackCollections))
-			{
-				AddCollectionsToList(item);
-			}
-		}
-
-
-		/// <summary>
-		/// Adds feedback colleciton to feedback collections
-		/// </summary>
-		/// <param name="feedbackCollection">BoolFeedback</param>
-		public void AddCollectionToList(FeedbackCollection<BoolFeedback> feedbackCollection)
-		{
-			foreach (var item in feedbackCollection.Where(item => item != null))
-			{
-				AddFeedbackToList(item);
-			}
-		}
-
-
-		/// <summary>
-		/// Adds feedback colleciton to feedback collections
-		/// </summary>
-		/// <param name="feedbackCollection">IntFeedback</param>
-		public void AddCollectionToList(FeedbackCollection<IntFeedback> feedbackCollection)
-		{
-			foreach (var item in feedbackCollection.Where(item => item != null))
-			{
-				AddFeedbackToList(item);
-			}
-		}
-
-
-		/// <summary>
-		/// Adds feedback colleciton to feedback collections
-		/// </summary>
-		/// <param name="feedbackCollection">StringFeedback</param>
-		public void AddCollectionToList(FeedbackCollection<StringFeedback> feedbackCollection)
-		{
-			foreach (var item in feedbackCollection.Where(item => item != null))
-			{
-				AddFeedbackToList(item);
-			}
-		}
-
-
-		/// <summary>
-		/// Adds individual feedbacks to feedback collection
-		/// </summary>
-		/// <param name="fb">Feedback</param>
-		public void AddFeedbackToList(PepperDash.Essentials.Core.Feedback fb)
-		{
-			if (fb == null || Feedbacks.Contains(fb)) return;
-
-			Feedbacks.Add(fb);
-		}
-
-
-		/// <summary>
-		/// Adds provided feedbacks to feedback collection list
-		/// </summary>
-		public void AddFeedbackCollecitons()
-		{
-			AddFeedbackToList(DeviceNameFeedback);
-			AddCollectionsToList(VideoInputSyncFeedbacks, InputHdcpEnableFeedback);
-			AddCollectionsToList(VideoOutputRouteFeedbacks);
-			AddCollectionsToList(InputNameFeedbacks, OutputNameFeedbacks, OutputRouteNameFeedback);
-		}
-
-
-		#endregion
-
-
 		#region Factory
 
 
@@ -608,7 +464,11 @@ namespace PepperDash_Essentials_DM.Chassis
 			}
 			public override EssentialsDevice BuildDevice(DeviceConfig dc)
 			{
-				Debug.Console(1, "Factory Attempting to create new HD-PSXxx device");
+				var key = dc.Key;
+				var name = dc.Name;
+				var type = dc.Type.ToLower();
+
+				Debug.Console(1, "Factory Attempting to create new {0} device", type);
 
 				var props = JsonConvert.DeserializeObject<HdPsXxxPropertiesConfig>(dc.Properties.ToString());
 				if (props == null)
@@ -617,12 +477,7 @@ namespace PepperDash_Essentials_DM.Chassis
 					return null;
 				}
 
-				var key = dc.Key;
+				var ipid = props.Control.IpIdInt;
-				var name = dc.Name;
-				var type = dc.Type.ToLower();
-				var control = props.Control;
-				var ipid = control.IpIdInt;
-				//var address = control.TcpSshProperties.Address;
 
 				switch (type)
 				{
@@ -653,5 +508,26 @@ namespace PepperDash_Essentials_DM.Chassis
 
 
 		#endregion
+
+		// TODO [ ] Implement CEC control
+		// return _chassis.HdmiDmLiteOutputs[0].DmLiteOutput.DmLiteOutputPort.StreamCec;
+		public Cec StreamCec
+		{
+			get
+			{
+				return _chassis.NumberOfOutputs == 1 
+					//? _chassis.HdmiDmLiteOutputs[0].DmLiteOutput.DmLiteOutputPort.StreamCec 
+					? _chassis.HdmiDmLiteOutputs[0].HdmiOutput.HdmiOutputPort.StreamCec
+					: null;
+			}
+		}
+
+		public IntFeedback HdmiInHdcpStateFeedback { get; private set; }
+		public void SetHdmiInHdcpState(eHdcpCapabilityType hdcpState)
+		{
+			throw new NotImplementedException();
+		}
+
+		public eHdcpCapabilityType HdmiInHdcpCapability { get; private set; }
 	}
 }