improved fix introduced by openhab#18168 (openhab#18284)

Signed-off-by: Alexander Friese <[email protected]>

Author: alexf2015

bundles/org.openhab.binding.easee/src/main/java/org/openhab/binding/easee/internal/handler/EaseeChargerHandler.java

@@ -39,6 +39,7 @@
 import org.openhab.core.thing.Thing;
 import org.openhab.core.thing.ThingStatus;
 import org.openhab.core.thing.ThingStatusDetail;
+import org.openhab.core.thing.ThingStatusInfo;
 import org.openhab.core.thing.binding.BaseThingHandler;
 import org.openhab.core.types.Command;
 import org.openhab.core.types.State;
@@ -70,10 +71,22 @@ public EaseeChargerHandler(Thing thing) {
         this.sessionDataPollingJobReference = new AtomicReference<>(null);
     }
 
-    void reInit() {
-        if (isInitialized()) {
-            dispose();
-            initialize();
+    @Override
+    public void bridgeStatusChanged(ThingStatusInfo bridgeStatusInfo) {
+        super.bridgeStatusChanged(bridgeStatusInfo);
+        if (bridgeStatusInfo.getStatus() == ThingStatus.ONLINE) {
+            logger.debug("bridgeStatusChanged: ONLINE");
+            if (isInitialized()) {
+                startPolling();
+            }
+        } else {
+            logger.debug("bridgeStatusChanged: NOT ONLINE");
+            if (isInitialized()) {
+                if (bridgeStatusInfo.getStatus() == ThingStatus.UNKNOWN) {
+                    updateStatus(ThingStatus.UNKNOWN, ThingStatusDetail.NONE, STATUS_WAITING_FOR_BRIDGE);
+                }
+                stopPolling();
+            }
         }
     }
 
@@ -131,6 +144,14 @@ private void startPolling() {
                 POLLING_INITIAL_DELAY, getBridgeConfiguration().getSessionDataPollingInterval(), TimeUnit.SECONDS));
     }
 
+    /**
+     * Stops the polling.
+     */
+    private void stopPolling() {
+        cancelJobReference(dataPollingJobReference);
+        cancelJobReference(sessionDataPollingJobReference);
+    }
+
     /**
      * Poll the Easee Cloud API one time.
      */
@@ -198,8 +219,7 @@ protected final void updateOnlineStatus(CommunicationStatus status, JsonObject j
     @Override
     public void dispose() {
         logger.debug("Handler disposed.");
-        cancelJobReference(dataPollingJobReference);
-        cancelJobReference(sessionDataPollingJobReference);
+        stopPolling();
     }
 
     /**

bundles/org.openhab.binding.easee/src/main/java/org/openhab/binding/easee/internal/handler/EaseeSiteHandler.java

@@ -35,7 +35,6 @@
 import org.openhab.binding.easee.internal.connector.CommunicationStatus;
 import org.openhab.binding.easee.internal.connector.WebInterface;
 import org.openhab.binding.easee.internal.discovery.EaseeSiteDiscoveryService;
-import org.openhab.core.config.core.Configuration;
 import org.openhab.core.config.discovery.DiscoveryService;
 import org.openhab.core.thing.Bridge;
 import org.openhab.core.thing.ThingStatus;
@@ -175,15 +174,6 @@ public void dispose() {
         webInterface.dispose();
     }
 
-    @Override
-    protected void updateConfiguration(Configuration configuration) {
-        super.updateConfiguration(configuration);
-        getChildChargerHandlers().forEach((name, handler) -> {
-            logger.debug("notify {}: config update", name);
-            handler.reInit();
-        });
-    }
-
     @Override
     public EaseeConfiguration getBridgeConfiguration() {
         return this.getConfigAs(EaseeConfiguration.class);