Erik/homebridge-kobold-Homaassistant-Flavour
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fixed a bug that caused the cleaning of all rooms at once

Browse Source
This commit is contained in:
Arne 2019-09-23 12:12:18 +02:00
parent 1a4308ac40
commit af54046927
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
accessories/neatoVacuumRobot.js
View File

@ -187,6 +187,7 @@ NeatoVacuumRobotAccessory.prototype = {
this.spotCleanAdvancedService.getCharacteristic(SpotRepeatCharacteristic).on('set', this.setSpotRepeat.bind(this));
this.spotCleanAdvancedService.getCharacteristic(SpotRepeatCharacteristic).on('get', this.getSpotRepeat.bind(this));
this.spotCleanAdvancedService.getCharacteristic(SpotWidthCharacteristic).on('set', this.setSpotWidth.bind(this));
this.spotCleanAdvancedService.getCharacteristic(SpotWidthCharacteristic).on('get', this.getSpotWidth.bind(this));
this.spotCleanAdvancedService.getCharacteristic(SpotHeightCharacteristic).on('set', this.setSpotHeight.bind(this));
this.spotCleanAdvancedService.getCharacteristic(SpotHeightCharacteristic).on('get', this.getSpotHeight.bind(this));
@ -262,6 +263,7 @@ NeatoVacuumRobotAccessory.prototype = {
setClean: function (on, callback, boundary)
{
debug(this.name + ": " + (on ? "Enabled " : "Disabled") + " Clean " + boundary);
this.platform.updateRobot(this.robot._serial, (error, result) =>
{
// Start
@ -658,7 +660,8 @@ NeatoVacuumRobotAccessory.prototype = {
if (!this.boundary)
{
// only update these values if the state is different from the current one, otherwise we might accidentally start an action
if (this.cleanService.getCharacteristic(Characteristic.On).value !== this.robot.canPause)
// AND dont set a clean switch to on if it's a room switch, otherwise we start the cleaning of each room at once
if (this.cleanService.getCharacteristic(Characteristic.On).value !== this.robot.canPause && (typeof this.boundary === 'undefined'))
{
this.cleanService.setCharacteristic(Characteristic.On, this.robot.canPause);
}