]> git.basschouten.com Git - openhab-addons.git/commitdiff
fix storage usage (#12305)
authoreugen <freiter@gmail.com>
Tue, 22 Feb 2022 05:50:19 +0000 (06:50 +0100)
committerGitHub <noreply@github.com>
Tue, 22 Feb 2022 05:50:19 +0000 (06:50 +0100)
Signed-off-by: Eugen Freiter <freiter@gmx.de>
bundles/org.openhab.io.homekit/src/main/java/org/openhab/io/homekit/internal/HomekitChangeListener.java
bundles/org.openhab.io.homekit/src/main/java/org/openhab/io/homekit/internal/HomekitImpl.java

index e3148fd1991199bb531bdd1c2561ce8c662bd72b..2a44a93b80a0cb46683ad53bdf9a1be4005a6985 100644 (file)
@@ -35,7 +35,6 @@ import org.openhab.core.items.Metadata;
 import org.openhab.core.items.MetadataKey;
 import org.openhab.core.items.MetadataRegistry;
 import org.openhab.core.storage.Storage;
-import org.openhab.core.storage.StorageService;
 import org.openhab.io.homekit.internal.accessories.HomekitAccessoryFactory;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -79,11 +78,11 @@ public class HomekitChangeListener implements ItemRegistryChangeListener {
     private final Debouncer applyUpdatesDebouncer;
 
     HomekitChangeListener(ItemRegistry itemRegistry, HomekitSettings settings, MetadataRegistry metadataRegistry,
-            StorageService storageService) {
+            Storage<String> storage) {
         this.itemRegistry = itemRegistry;
         this.settings = settings;
         this.metadataRegistry = metadataRegistry;
-        storage = storageService.getStorage(HomekitAuthInfoImpl.STORAGE_KEY);
+        this.storage = storage;
         this.applyUpdatesDebouncer = new Debouncer("update-homekit-devices", scheduler, Duration.ofMillis(1000),
                 Clock.systemUTC(), this::applyUpdates);
         metadataChangeListener = new RegistryChangeListener<Metadata>() {
index 77c78a8fc0e826aad703d510208af8bb3b332b07..7fa41890faeb232a27cf34d5bf14b37bab1d562c 100644 (file)
@@ -37,6 +37,7 @@ import org.openhab.core.items.MetadataRegistry;
 import org.openhab.core.net.CidrAddress;
 import org.openhab.core.net.NetworkAddressChangeListener;
 import org.openhab.core.net.NetworkAddressService;
+import org.openhab.core.storage.Storage;
 import org.openhab.core.storage.StorageService;
 import org.openhab.io.homekit.Homekit;
 import org.osgi.framework.Constants;
@@ -70,6 +71,7 @@ public class HomekitImpl implements Homekit, NetworkAddressChangeListener {
 
     private final NetworkAddressService networkAddressService;
     private final ConfigurationAdmin configAdmin;
+    private final Storage<String> storage;
 
     private HomekitAuthInfoImpl authInfo;
     private HomekitSettings settings;
@@ -92,11 +94,11 @@ public class HomekitImpl implements Homekit, NetworkAddressChangeListener {
         this.configAdmin = configAdmin;
         this.settings = processConfig(properties);
         this.mdnsClient = mdnsClient;
+        this.storage = storageService.getStorage(HomekitAuthInfoImpl.STORAGE_KEY);
         networkAddressService.addNetworkAddressChangeListener(this);
-        this.changeListener = new HomekitChangeListener(itemRegistry, settings, metadataRegistry, storageService);
+        this.changeListener = new HomekitChangeListener(itemRegistry, settings, metadataRegistry, storage);
         try {
-            authInfo = new HomekitAuthInfoImpl(storageService.getStorage(HomekitAuthInfoImpl.STORAGE_KEY), settings.pin,
-                    settings.setupId, settings.blockUserDeletion);
+            authInfo = new HomekitAuthInfoImpl(storage, settings.pin, settings.setupId, settings.blockUserDeletion);
             startHomekitServer();
         } catch (IOException | InvalidAlgorithmParameterException e) {
             logger.warn("cannot activate HomeKit binding. {}", e.getMessage());