Merge pull request #8907 from huffman/fix/away-annoying-log-messages

Fix away.js logging all incoming Messages
This commit is contained in:
Brad Hefta-Gaub 2016-10-25 17:39:07 -07:00 committed by GitHub
commit 9d23fbeafb

View file

@ -304,9 +304,9 @@ function setEnabled(value) {
var CHANNEL_AWAY_ENABLE = "Hifi-Away-Enable";
var handleMessage = function(channel, message, sender) {
print("Got away message");
if (channel == CHANNEL_AWAY_ENABLE) {
setEnabled(message == 'enable');
if (channel === CHANNEL_AWAY_ENABLE) {
print("away.js | Got message on Hifi-Away-Enable: ", message);
setEnabled(message === 'enable');
}
}
Messages.subscribe(CHANNEL_AWAY_ENABLE);
@ -343,6 +343,7 @@ Script.scriptEnding.connect(function () {
Controller.disableMapping(eventMappingName);
Controller.mousePressEvent.disconnect(goActive);
Controller.keyPressEvent.disconnect(maybeGoActive);
Messages.messageReceived.disconnect(handleMessage);
});
if (HMD.active && !HMD.mounted) {