[thing] Add toString overrides to ThingImpl & BridgeImpl (#4382)

Signed-off-by: Florian Hotze <[email protected]>
openhab · Sep 19, 2024 · 437a885 · 437a885
1 parent 5f14cd1
commit 437a885
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 0 deletions.
diff --git a/bundles/org.openhab.core.thing/src/main/java/org/openhab/core/thing/internal/BridgeImpl.java b/bundles/org.openhab.core.thing/src/main/java/org/openhab/core/thing/internal/BridgeImpl.java
@@ -89,4 +89,9 @@ public List<Thing> getThings() {
         }
         return bridgeHandler;
     }
+
+    @Override
+    public String toString() {
+        return super.toString().replace("Bridge=False", "Bridge=True");
+    }
 }
diff --git a/bundles/org.openhab.core.thing/src/main/java/org/openhab/core/thing/internal/ThingImpl.java b/bundles/org.openhab.core.thing/src/main/java/org/openhab/core/thing/internal/ThingImpl.java
@@ -237,6 +237,31 @@ public boolean isEnabled() {
         return ThingStatusDetail.DISABLED != getStatusInfo().getStatusDetail();
     }
 
+    @Override
+    public String toString() {
+        // Configuration is deliberately excluded because it might include sensitive data like passwords.
+        StringBuilder sb = new StringBuilder(getUID().toString());
+        sb.append(" (ThingTypeUID=");
+        sb.append(getThingTypeUID());
+        sb.append(", Bridge=False");
+        if (getBridgeUID() != null) {
+            sb.append(", BridgeUID=");
+            sb.append(getBridgeUID());
+        }
+        sb.append(", Label=");
+        sb.append(getLabel());
+        if (getLocation() != null) {
+            sb.append(", Location=");
+            sb.append(getLocation());
+        }
+        sb.append(", Status=");
+        sb.append(getStatus());
+        sb.append(", StatusInfo=");
+        sb.append(getStatusInfo());
+        sb.append(")");
+        return sb.toString();
+    }
+
     @Override
     public int hashCode() {
         final int prime = 31;