From 2bfba0a4f156c943e1e909e962565c9a74c6cc17 Mon Sep 17 00:00:00 2001 From: Matt Date: Fri, 31 May 2024 04:13:09 -0400 Subject: [PATCH] [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 --- bundles/org.openhab.binding.haywardomnilogic/README.md | 3 +++ .../internal/handler/HaywardChlorinatorHandler.java | 1 + 2 files changed, 4 insertions(+) diff --git a/bundles/org.openhab.binding.haywardomnilogic/README.md b/bundles/org.openhab.binding.haywardomnilogic/README.md index 950939661b057..2508d59ae300a 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 3c67d568bb0f6..a315302acaa8f 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 void getTelemetry(String xmlResponse) throws HaywardException { // 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);