From: truidix <49732750+truidix@users.noreply.github.com>
Date: Thu, 21 Jul 2022 19:31:02 +0000 (+0200)
Subject: check if negativeSwitch is "on" and positiveSwitch is "off. If so: just (#13152)
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=831e7ec257493c62c5d546698fee761d4c2a7acf;p=openhab-addons.git

check if negativeSwitch is "on" and positiveSwitch is "off. If so: just (#13152)

print a warning instead of throwing a exception

Signed-off-by: David Kumar <git@truidix.de>
---

diff --git a/bundles/org.openhab.binding.km200/src/main/java/org/openhab/binding/km200/internal/handler/KM200SwitchProgramServiceHandler.java b/bundles/org.openhab.binding.km200/src/main/java/org/openhab/binding/km200/internal/handler/KM200SwitchProgramServiceHandler.java
index cfd5cad902..9a16145ad2 100644
--- a/bundles/org.openhab.binding.km200/src/main/java/org/openhab/binding/km200/internal/handler/KM200SwitchProgramServiceHandler.java
+++ b/bundles/org.openhab.binding.km200/src/main/java/org/openhab/binding/km200/internal/handler/KM200SwitchProgramServiceHandler.java
@@ -101,7 +101,8 @@ public class KM200SwitchProgramServiceHandler {
         if (!setpoints.contains(setpoint)) {
             if (setpoints.size() == 2 && "on".compareTo(setpoint) == 0) {
                 if ("high".compareTo(setpoints.get(0)) == 0 && "off".compareTo(setpoints.get(1)) == 0) {
-                    if ("on".compareTo(positiveSwitch) == 0 && "off".compareTo(negativeSwitch) == 0) {
+                    if (("on".compareTo(positiveSwitch) == 0 && "off".compareTo(negativeSwitch) == 0)
+                            || ("off".compareTo(positiveSwitch) == 0 && "on".compareTo(negativeSwitch) == 0)) {
                         logger.info(
                                 "!!! Wrong configuration on device. 'on' instead of 'high' in switch program. It seems that's a firmware problem-> ignoring it !!!");
                     } else {