Don't loop to check if MoH is needed

Instead, start it once when you join the room, and check each time a peer leaves if it must be re-enabled

public/js/vroom.js

@@ -589,6 +589,17 @@ function initVroom(room) {
     peers.local.videoPaused = false;
   }
 
+  // Check if MoH is needed
+  function checkMoh(){
+    if (!moh && Object.keys(peers).length < 2){
+      if (!$('#pauseMohButton').is(':checked')){
+        $('#mohPlayer')[0].play();
+        moh = true;
+      }
+      $('.aloneEl').show(200);
+    }
+  }
+
   // An owner is muting/unmuting ourself
   webrtc.on('owner_toggle_mute', function(data){
     // Ignore this if the remote peer isn't the owner of the room
@@ -912,16 +923,7 @@ function initVroom(room) {
         }
       }
     });
-    // We will check if moh is needed
+    checkMoh();
-    setInterval(function(){
-      if (!moh && Object.keys(peers).length < 2){
-        if (!$('#pauseMohButton').is(':checked')){
-          $('#mohPlayer')[0].play();
-          moh = true;
-        }
-        $('.aloneEl').show(200);
-      }
-    }, 2000);
     $('#videoLocalContainer').show(200);
   });
 
@@ -964,6 +966,7 @@ function initVroom(room) {
       }, 500);
       mainVid = false;
     }
+    checkMoh();
   });
 
   // Error sending something through dataChannel