fix: subscribe to VU meters upon connection

olzzon · Nov 12, 2020 · bf5fce7 · bf5fce7
1 parent c518510
commit bf5fce7
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 4 deletions.
diff --git a/client/index.tsx b/client/index.tsx
@@ -57,9 +57,6 @@ socketClientHandlers()
 window.socketIoClient.emit('get-store', 'update local store')
 window.socketIoClient.emit('get-settings', 'update local settings')
 window.socketIoClient.emit('get-mixerprotocol', 'get selected mixerprotocol')
-if (!window.location.search.includes('vu=0')) {
-    window.socketIoClient.emit('subscribe-vu-meter', 'subscribe to vu meters')
-}
 
 ReactDom.render(
     <ReduxProvider store={storeRedux}>

diff --git a/client/utils/SocketClientHandlers.ts b/client/utils/SocketClientHandlers.ts
@@ -36,6 +36,13 @@ export const socketClientHandlers = () => {
         .on('connect', () => {
             window.storeRedux.dispatch(storeSetServerOnline(true))
             console.log('CONNECTED TO SISYFOS SERVER')
+            if (!window.location.search.includes('vu=0')) {
+                // subscribe to VU'
+                window.socketIoClient.emit(
+                    'subscribe-vu-meter',
+                    'subscribe to vu meters'
+                )
+            }
         })
         .on('disconnect', () => {
             window.storeRedux.dispatch(storeSetServerOnline(false))

diff --git a/server/utils/vuServer.ts b/server/utils/vuServer.ts
@@ -6,7 +6,10 @@ export enum VuType {
 const sockets: Array<any> = []
 
 export function socketSubscribeVu(socket: any) {
-    sockets.push(socket)
+    const i = sockets.indexOf(socket)
+    if (i === -1) {
+        sockets.push(socket)
+    }
 }
 
 export function socketUnsubscribeVu(socket: any) {