[deconz] Fix Thermostat setting target temperature (#9189)

Improve logging on sending commands by checking http response status
code

Signed-off-by: Lukas Agethen <lukas83@gmx.de>

diff --git a/bundles/org.openhab.binding.deconz/src/main/java/org/openhab/binding/deconz/internal/handler/DeconzBaseThingHandler.java b/bundles/org.openhab.binding.deconz/src/main/java/org/openhab/binding/deconz/internal/handler/DeconzBaseThingHandler.java
index 9676e07e28fc0ccadc0be02c0b182a20d3918fb5..6e310ea8d372f46dafd541953e1fb14d8c9a0dec 100644 (file)
--- a/bundles/org.openhab.binding.deconz/src/main/java/org/openhab/binding/deconz/internal/handler/DeconzBaseThingHandler.java
+++ b/bundles/org.openhab.binding.deconz/src/main/java/org/openhab/binding/deconz/internal/handler/DeconzBaseThingHandler.java
@@ -194,9 +194,14 @@ public abstract class DeconzBaseThingHandler<T extends DeconzBaseMessage> extend
             if (acceptProcessing != null) {
                 acceptProcessing.run();
             }
-            logger.trace("Result code={}, body={}", v.getResponseCode(), v.getBody());
+            if (v.getResponseCode() != java.net.HttpURLConnection.HTTP_OK) {
+                logger.warn("Sending command {} to channel {} failed: {} - {}", originalCommand, channelUID,
+                        v.getResponseCode(), v.getBody());
+            } else {
+                logger.trace("Result code={}, body={}", v.getResponseCode(), v.getBody());
+            }
         }).exceptionally(e -> {
-            logger.debug("Sending command {} to channel {} failed: {} - {}", originalCommand, channelUID, e.getClass(),
+            logger.warn("Sending command {} to channel {} failed: {} - {}", originalCommand, channelUID, e.getClass(),
                     e.getMessage());
             return null;
         });

diff --git a/bundles/org.openhab.binding.deconz/src/main/java/org/openhab/binding/deconz/internal/types/ResourceType.java b/bundles/org.openhab.binding.deconz/src/main/java/org/openhab/binding/deconz/internal/types/ResourceType.java
index 6a33ad00a42a588ff00088dde4a3f70a29defa94..02ddf78c49389064f4e7473ebc2603d373038414 100644 (file)
--- a/bundles/org.openhab.binding.deconz/src/main/java/org/openhab/binding/deconz/internal/types/ResourceType.java
+++ b/bundles/org.openhab.binding.deconz/src/main/java/org/openhab/binding/deconz/internal/types/ResourceType.java
@@ -29,7 +29,7 @@ import org.slf4j.LoggerFactory;
 public enum ResourceType {
     GROUPS("groups", "action"),
     LIGHTS("lights", "state"),
-    SENSORS("sensors", ""),
+    SENSORS("sensors", "config"),
     UNKNOWN("", "");
 
     private static final Map<String, ResourceType> MAPPING = Arrays.stream(ResourceType.values())