mtan93/homebridge
1
0
Fork 0
mirror of https://github.com/mtan93/homebridge.git synced 2026-04-17 14:23:07 +01:00
Code Issues Packages Projects Releases Wiki Activity

Guard against multiple WeMo callbacks

Browse Source 
Closes #296
This commit is contained in:
Nick Farina
2015-11-03 06:37:53 -08:00
parent 108602a6b0
commit eabf45526d
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
accessories/WeMo.js
View File

@@ -89,7 +89,15 @@ WeMoAccessory.prototype.setPowerOn = function(powerOn, callback) {
var binaryState = powerOn ? 1 : 0; // wemo langauge var binaryState = powerOn ? 1 : 0; // wemo langauge
this.log("Setting power state on the '%s' to %s", this.wemoName, binaryState); this.log("Setting power state on the '%s' to %s", this.wemoName, binaryState);
var callbackWasCalled = false;
this.device.setBinaryState(binaryState, function(err, result) { this.device.setBinaryState(binaryState, function(err, result) {
if (callbackWasCalled) {
this.log("WARNING: setBinaryState called its callback more than once! Discarding the second one.");
}
callbackWasCalled = true;
if (!err) { if (!err) {
this.log("Successfully set power state on the '%s' to %s", this.wemoName, binaryState); this.log("Successfully set power state on the '%s' to %s", this.wemoName, binaryState);
callback(null); callback(null);