fix(InputPropertiesCommand): include the inputId of each source in th…

…e state tree
nrkno · Apr 30, 2018 · 3fdabba · Julusian · May 1, 2018 · 3fdabba
1 parent 2199e5c
commit 3fdabba
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/src/commands/Inputs/InputPropertiesCommand.ts b/src/commands/Inputs/InputPropertiesCommand.ts
@@ -41,6 +41,7 @@ export class InputPropertiesCommand extends AbstractCommand {
 		}
 
 		this.properties = {
+			inputId: rawCommand.readUInt16BE(0),
 			longName: Util.bufToNullTerminatedString(rawCommand, 2, 20),
 			shortName: Util.bufToNullTerminatedString(rawCommand, 22, 4),
 			externalPorts: externalPorts.length > 0 ? externalPorts : null,
@@ -63,6 +64,9 @@ export class InputPropertiesCommand extends AbstractCommand {
 	}
 
 	applyToState (state: AtemState) {
+		// @TODO(Lange - 04/30/2018): We may need something to clean up inputs which
+		// don't exist anymore, which can happen when switching the connection from
+		// one model of ATEM to another.
 		state.inputs[this.inputId] = {
 			...this.properties
 		}

diff --git a/src/state/input.ts b/src/state/input.ts
@@ -6,6 +6,7 @@ import {
 } from '../enums'
 
 export interface InputChannel {
+	readonly inputId: number,
 	longName: string
 	shortName: string
 	isExternal: boolean