check if negativeSwitch is "on" and positiveSwitch is "off. If so: ju…

…st (openhab#13152) print a warning instead of throwing a exception Signed-off-by: David Kumar <[email protected]>
Hilbrand · Jul 21, 2022 · 831e7ec · 831e7ec
1 parent 0577ee8
commit 831e7ec
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...ain/java/org/openhab/binding/km200/internal/handler/KM200SwitchProgramServiceHandler.java b/...ain/java/org/openhab/binding/km200/internal/handler/KM200SwitchProgramServiceHandler.java
@@ -101,7 +101,8 @@ void addSwitch(String day, String setpoint, int time) {
         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 {