From 223c9f929b077be04b2b5d2fc47bd099033afab0 Mon Sep 17 00:00:00 2001 From: Fabian Wolter Date: Wed, 23 Feb 2022 15:23:19 +0100 Subject: [PATCH] [hdpowerview] Let the framework change the child Thing statuses (#12330) Signed-off-by: Fabian Wolter --- .../internal/handler/HDPowerViewHubHandler.java | 1 + .../internal/handler/HDPowerViewRepeaterHandler.java | 9 +++------ .../internal/handler/HDPowerViewShadeHandler.java | 7 +------ 3 files changed, 5 insertions(+), 12 deletions(-) diff --git a/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewHubHandler.java b/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewHubHandler.java index 7da07c21f..477ebdb73 100644 --- a/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewHubHandler.java +++ b/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewHubHandler.java @@ -164,6 +164,7 @@ public class HDPowerViewHubHandler extends BaseBridgeHandler { return; } + updateStatus(ThingStatus.UNKNOWN); pendingShadeInitializations.clear(); webTargets = new HDPowerViewWebTargets(httpClient, host); refreshInterval = config.refresh; diff --git a/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewRepeaterHandler.java b/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewRepeaterHandler.java index 1274ff1fa..a3a0cf35f 100644 --- a/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewRepeaterHandler.java +++ b/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewRepeaterHandler.java @@ -82,12 +82,9 @@ public class HDPowerViewRepeaterHandler extends AbstractHubbedThingHandler { "@text/offline.conf-error.invalid-bridge-handler"); return; } - ThingStatus bridgeStatus = bridge.getStatus(); - if (bridgeStatus == ThingStatus.ONLINE) { - updateStatus(ThingStatus.UNKNOWN); - } else { - updateStatus(ThingStatus.OFFLINE, ThingStatusDetail.BRIDGE_OFFLINE); - } + + updateStatus(ThingStatus.UNKNOWN); + scheduleRefreshJob(); } diff --git a/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewShadeHandler.java b/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewShadeHandler.java index 58c619508..5b0cb829a 100644 --- a/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewShadeHandler.java +++ b/bundles/org.openhab.binding.hdpowerview/src/main/java/org/openhab/binding/hdpowerview/internal/handler/HDPowerViewShadeHandler.java @@ -109,12 +109,7 @@ public class HDPowerViewShadeHandler extends AbstractHubbedThingHandler { "@text/offline.conf-error.invalid-bridge-handler"); return; } - ThingStatus bridgeStatus = bridge.getStatus(); - if (bridgeStatus == ThingStatus.ONLINE) { - updateStatus(ThingStatus.UNKNOWN); - } else { - updateStatus(ThingStatus.OFFLINE, ThingStatusDetail.BRIDGE_OFFLINE); - } + updateStatus(ThingStatus.UNKNOWN); } @Override