demus/openhab-addons
1
0
Fork 0
mirror of https://github.com/openhab/openhab-addons.git synced 2025-01-10 23:22:02 +01:00
Code Issues Packages Projects Releases Wiki Activity

[openwebnet] Do not store dimmer brightness if already 0 (#14822)

Browse Source 
Fixes #14785

Signed-off-by: Massimo Valla <mvcode00@gmail.com>
This commit is contained in:
M Valla 2023-04-21 13:49:07 +02:00 committed by GitHub
parent cb4d232328
commit 02fd67f6ce
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
bundles/org.openhab.binding.openwebnet/src/main/java/org/openhab/binding/openwebnet/internal/handler/OpenWebNetLightingHandler.java
View File

@ -194,13 +194,16 @@ public class OpenWebNetLightingHandler extends OpenWebNetThingHandler {
private void dimLightTo(int percent, Command command) {
logger.debug(" DIM dimLightTo({}) bri={} briBeforeOff={}", percent, brightness, brightnessBeforeOff);
int newBrightness = percent;
if (newBrightness == UNKNOWN_STATE) {
// we do not know last brightness -> set dimmer to 100%
if (newBrightness == UNKNOWN_STATE) { // we do not know last brightness -> set dimmer to 100%
newBrightness = 100;
} else if (newBrightness <= 0) {
newBrightness = 0;
brightnessBeforeOff = brightness;
logger.debug(" DIM saved bri before sending bri=0 command to device");
if (brightness == 0) {
logger.debug(" DIM bri already 0: no need to save it");
} else {
brightnessBeforeOff = brightness;
logger.debug(" DIM saved briBeforeOff={} before sending bri=0 command to device",
brightnessBeforeOff);
}
} else if (newBrightness > 100) {
newBrightness = 100;
}
@ -280,8 +283,7 @@ public class OpenWebNetLightingHandler extends OpenWebNetThingHandler {
return;
} else {
// otherwise we interpret this ON event as the requestStatus response event with
// level=1
// so we proceed to call updateBrightnessState()
// level=1 so we proceed to call updateBrightnessState()
logger.debug(" $BRI 'ON' is the requestStatus response level");
}
}