fix: X32 - mixer was offline when receiving data for a channel with a…

… higher assigned fader than total amount of faders
olzzon · Jun 8, 2020 · 0a38821 · 0a38821
1 parent 252face
commit 0a38821
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/server/reducers/fadersReducer.ts b/server/reducers/fadersReducer.ts
@@ -128,6 +128,9 @@ export const faders = (
             fader: [...state[0].fader],
         },
     ]
+    if (action.channel > nextState[0].fader.length) {
+        return nextState
+    }
 
     switch (action.type) {
         case SET_VU_LEVEL: //channel:  level:

diff --git a/server/utils/mixerConnections/OscMixerConnection.ts b/server/utils/mixerConnections/OscMixerConnection.ts
@@ -445,10 +445,8 @@ export class OscMixerConnection {
                 }
             })
             .on('error', (error: any) => {
-                this.mixerOnline(false)
                 global.mainThreadHandler.updateFullClientStore()
                 logger.error('Error : ' + String(error), {})
-                logger.info('Lost OSC connection', {})
             })
             .on('disconnect', () => {
                 this.mixerOnline(false)