From: Marcel <marcelrv@users.noreply.github.com>
Date: Tue, 6 Oct 2020 19:45:10 +0000 (+0200)
Subject: [miio] onoffpara commandtype fix (#8675)
X-Git-Url: https://git.basschouten.com/?a=commitdiff_plain;h=43be10c2047d27d7950eafdb3efab6e57b6f2ff9;p=openhab-addons.git

[miio] onoffpara commandtype fix (#8675)

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
---

diff --git a/bundles/org.openhab.binding.miio/src/main/java/org/openhab/binding/miio/internal/handler/MiIoBasicHandler.java b/bundles/org.openhab.binding.miio/src/main/java/org/openhab/binding/miio/internal/handler/MiIoBasicHandler.java
index 4ba8cae7f6..81824519ae 100644
--- a/bundles/org.openhab.binding.miio/src/main/java/org/openhab/binding/miio/internal/handler/MiIoBasicHandler.java
+++ b/bundles/org.openhab.binding.miio/src/main/java/org/openhab/binding/miio/internal/handler/MiIoBasicHandler.java
@@ -152,6 +152,7 @@ public class MiIoBasicHandler extends MiIoAbstractHandler {
                             value = new JsonPrimitive(command == OnOffType.ON ? "on" : "off");
                         } else if (paramType == CommandParameterType.ONOFFPARA) {
                             cmd = cmd.replace("*", command == OnOffType.ON ? "on" : "off");
+                            value = new JsonArray();
                         } else if (paramType == CommandParameterType.ONOFFBOOL) {
                             boolean boolCommand = command == OnOffType.ON;
                             value = new JsonPrimitive(boolCommand);
@@ -186,7 +187,8 @@ public class MiIoBasicHandler extends MiIoAbstractHandler {
                             value = miotTransform(miIoBasicChannel, value);
                         }
                     }
-                    if (paramType != CommandParameterType.NONE && value != null) {
+                    if (paramType != CommandParameterType.NONE && paramType != CommandParameterType.ONOFFPARA
+                            && value != null) {
                         if (parameters.size() > 0) {
                             parameters.set(valuePos, value);
                         } else {