[fronius] Fix bridge config update not properly handled (#18375)

Fixes #18123.

Signed-off-by: Florian Hotze <[email protected]>

Author: florian-h05

bundles/org.openhab.binding.fronius/src/main/java/org/openhab/binding/fronius/internal/handler/FroniusBaseThingHandler.java

@@ -14,6 +14,8 @@
 
 import static org.openhab.binding.fronius.internal.FroniusBindingConstants.API_TIMEOUT;
 
+import java.util.Map;
+
 import org.eclipse.jdt.annotation.NonNullByDefault;
 import org.eclipse.jdt.annotation.Nullable;
 import org.eclipse.jetty.http.HttpMethod;
@@ -84,6 +86,14 @@ public void initialize() {
         }
     }
 
+    /**
+     * Called by the bridge to handle a bridge configuration update.
+     *
+     * @param configurationParameters map of changed bridge configuration parameters
+     */
+    protected void handleBridgeConfigurationUpdate(Map<String, Object> configurationParameters) {
+    }
+
     /**
      * Update all Channels
      */

bundles/org.openhab.binding.fronius/src/main/java/org/openhab/binding/fronius/internal/handler/FroniusBridgeHandler.java

@@ -15,6 +15,7 @@
 import static org.openhab.binding.fronius.internal.FroniusBindingConstants.API_TIMEOUT;
 
 import java.util.HashSet;
+import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.ScheduledFuture;
 import java.util.concurrent.TimeUnit;
@@ -95,6 +96,15 @@ public void dispose() {
         }
     }
 
+    @Override
+    public void handleConfigurationUpdate(Map<String, Object> configurationParameters) {
+        super.handleConfigurationUpdate(configurationParameters);
+
+        for (FroniusBaseThingHandler service : services) {
+            service.handleBridgeConfigurationUpdate(configurationParameters);
+        }
+    }
+
     @Override
     public void childHandlerInitialized(ThingHandler childHandler, Thing childThing) {
         if (childHandler instanceof FroniusBaseThingHandler handler) {

bundles/org.openhab.binding.fronius/src/main/java/org/openhab/binding/fronius/internal/handler/FroniusSymoInverterHandler.java

@@ -16,6 +16,7 @@
 import java.time.LocalTime;
 import java.util.Collection;
 import java.util.List;
+import java.util.Map;
 import java.util.Optional;
 
 import javax.measure.Unit;
@@ -85,14 +86,20 @@ protected void handleRefresh(FroniusBridgeConfiguration bridgeConfiguration) thr
         updateChannels();
     }
 
+    private void initializeBatteryControl(String hostname, String username, String password) {
+        if (hostname != null && username != null && password != null) {
+            batteryControl = new FroniusBatteryControl(httpClient, URI.create("http://" + hostname + "/"), username,
+                    password);
+        } else {
+            batteryControl = null;
+        }
+    }
+
     @Override
     public void initialize() {
         config = getConfigAs(FroniusBaseDeviceConfiguration.class);
         FroniusBridgeConfiguration bridgeConfig = getBridge().getConfiguration().as(FroniusBridgeConfiguration.class);
-        if (bridgeConfig.username != null && bridgeConfig.password != null) {
-            batteryControl = new FroniusBatteryControl(httpClient, URI.create("http://" + bridgeConfig.hostname + "/"),
-                    bridgeConfig.username, bridgeConfig.password);
-        }
+        initializeBatteryControl(bridgeConfig.hostname, bridgeConfig.username, bridgeConfig.password);
         super.initialize();
     }
 
@@ -101,6 +108,13 @@ public Collection<Class<? extends ThingHandlerService>> getServices() {
         return List.of(FroniusSymoInverterActions.class);
     }
 
+    @Override
+    public void handleBridgeConfigurationUpdate(Map<String, Object> configurationParameters) {
+        super.handleBridgeConfigurationUpdate(configurationParameters);
+        FroniusBridgeConfiguration bridgeConfig = getBridge().getConfiguration().as(FroniusBridgeConfiguration.class);
+        initializeBatteryControl(bridgeConfig.hostname, bridgeConfig.username, bridgeConfig.password);
+    }
+
     private @Nullable FroniusBatteryControl getBatteryControl() {
         if (batteryControl == null) {
             logger.warn("Battery control is not available. Check the bridge configuration.");