From: Matthew Skinner <matt@pcmus.com>
Date: Wed, 15 Dec 2021 10:30:21 +0000 (+1100)
Subject: Fix two events being logged when off. (#11786)
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=88d74b72eab87985ae35550606f3906c6129acc6;p=openhab-addons.git

Fix two events being logged when off. (#11786)

Signed-off-by: Matthew Skinner <matt@pcmus.com>
---

diff --git a/bundles/org.openhab.binding.wled/src/main/java/org/openhab/binding/wled/internal/api/WledApiV084.java b/bundles/org.openhab.binding.wled/src/main/java/org/openhab/binding/wled/internal/api/WledApiV084.java
index bd6d00ddef..2053928e67 100644
--- a/bundles/org.openhab.binding.wled/src/main/java/org/openhab/binding/wled/internal/api/WledApiV084.java
+++ b/bundles/org.openhab.binding.wled/src/main/java/org/openhab/binding/wled/internal/api/WledApiV084.java
@@ -264,7 +264,6 @@ public class WledApiV084 implements WledApi {
         }
         HSBType tempHSB = WLedHelper
                 .parseToHSBType(state.stateResponse.seg[handler.config.segmentIndex].col[0].toString());
-        handler.update(CHANNEL_MASTER_CONTROLS, tempHSB);
         handler.update(CHANNEL_PRIMARY_COLOR, tempHSB);
         handler.update(CHANNEL_SECONDARY_COLOR,
                 WLedHelper.parseToHSBType(state.stateResponse.seg[handler.config.segmentIndex].col[1].toString()));
@@ -283,6 +282,7 @@ public class WledApiV084 implements WledApi {
             handler.update(CHANNEL_MASTER_CONTROLS, OnOffType.OFF);
             handler.update(CHANNEL_SEGMENT_BRIGHTNESS, OnOffType.OFF);
         } else {
+            handler.update(CHANNEL_MASTER_CONTROLS, tempHSB);
             handler.update(CHANNEL_SEGMENT_BRIGHTNESS,
                     new PercentType(new BigDecimal(state.stateResponse.seg[handler.config.segmentIndex].bri)
                             .divide(BIG_DECIMAL_2_55, RoundingMode.HALF_UP)));