[lgwebos] Update the thing configuration only if the key is different (…

…openhab#7302) * [lgwebos] Update the thing configuration only if the key is different Avoid concurrent edit of thing configuration/properties Signed-off-by: Laurent Garnier <[email protected]> Signed-off-by: Hans-Reiner Hoffmann <[email protected]>
Hans-Reiner · Apr 11, 2020 · 81e6514 · 81e6514
1 parent b0643c2
commit 81e6514
Showing 1 changed file with 11 additions and 7 deletions.
diff --git a/...ng.lgwebos/src/main/java/org/openhab/binding/lgwebos/internal/handler/LGWebOSHandler.java b/...ng.lgwebos/src/main/java/org/openhab/binding/lgwebos/internal/handler/LGWebOSHandler.java
@@ -242,17 +242,21 @@ public String getKey() {
 
     @Override
     public void storeKey(@Nullable String key) {
-        // store it current configuration and avoiding complete re-initialization via handleConfigurationUpdate
-        getLGWebOSConfig().key = key;
-
-        // persist the configuration change
-        Configuration configuration = editConfiguration();
-        configuration.put(LGWebOSBindingConstants.CONFIG_KEY, key);
-        updateConfiguration(configuration);
+        if (!getKey().equals(key)) {
+            logger.debug("store new key");
+            // store it current configuration and avoiding complete re-initialization via handleConfigurationUpdate
+            getLGWebOSConfig().key = key;
+
+            // persist the configuration change
+            Configuration configuration = editConfiguration();
+            configuration.put(LGWebOSBindingConstants.CONFIG_KEY, key);
+            updateConfiguration(configuration);
+        }
     }
 
     @Override
     public void storeProperties(Map<String, String> properties) {
+        logger.debug("storeProperties {}", properties);
         Map<String, String> map = editProperties();
         map.putAll(properties);
         updateProperties(map);