From: Matt <mmyers75@icloud.com>
Date: Fri, 31 May 2024 08:13:09 +0000 (-0400)
Subject: [haywardomnilogic] Fixed chlorinator bug (#16820)
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=2bfba0a4f156c943e1e909e962565c9a74c6cc17;p=openhab-addons.git

[haywardomnilogic] Fixed chlorinator bug (#16820)

* Fixed chlorinator handler bug that caused the chlorinator to turn off if the chlorinator salt % output channel was adjusted.

Signed-off-by: Matt Myers <mmyers75@icloud.com>
---

diff --git a/bundles/org.openhab.binding.haywardomnilogic/README.md b/bundles/org.openhab.binding.haywardomnilogic/README.md
index 950939661b..2508d59ae3 100644
--- a/bundles/org.openhab.binding.haywardomnilogic/README.md
+++ b/bundles/org.openhab.binding.haywardomnilogic/README.md
@@ -120,6 +120,9 @@ Hayward OmniLogic Connection Parameters:
 | colorLogicLightEnable      | Switch    | Colorlogic Light enable       |     R/W    |
 | colorLogicLightState       | String    | Colorlogic Light state        |      R     |
 | colorLogicLightCurrentShow | String    | Colorlogic Light current show |     R/W    |
+| colorLogicLightBrightness  | String    | Colorlogic Light brightness   |     R/W    |
+| colorLogicLightSpeed       | String    | Colorlogic Light speed        |     R/W    |
+**Brightness and speed channels only available on Hayward V2 lights
 
 ### Filter Channels
 
diff --git a/bundles/org.openhab.binding.haywardomnilogic/src/main/java/org/openhab/binding/haywardomnilogic/internal/handler/HaywardChlorinatorHandler.java b/bundles/org.openhab.binding.haywardomnilogic/src/main/java/org/openhab/binding/haywardomnilogic/internal/handler/HaywardChlorinatorHandler.java
index 3c67d568bb..a315302aca 100644
--- a/bundles/org.openhab.binding.haywardomnilogic/src/main/java/org/openhab/binding/haywardomnilogic/internal/handler/HaywardChlorinatorHandler.java
+++ b/bundles/org.openhab.binding.haywardomnilogic/src/main/java/org/openhab/binding/haywardomnilogic/internal/handler/HaywardChlorinatorHandler.java
@@ -61,6 +61,7 @@ public class HaywardChlorinatorHandler extends HaywardThingHandler {
                     // Enable
                     data = bridgehandler.evaluateXPath("//Chlorinator/@enable", xmlResponse);
                     updateData(HaywardBindingConstants.CHANNEL_CHLORINATOR_ENABLE, data.get(i));
+                    channelStates.putAll(updateData(HaywardBindingConstants.CHANNEL_CHLORINATOR_ENABLE, data.get(i)));
 
                     // Operating Mode
                     data = bridgehandler.evaluateXPath("//Chlorinator/@operatingMode", xmlResponse);