Memory persistence service should rely on assigned persistence servic…

…e id. Signed-off-by: Łukasz Dywicki <luke@code-house.org>
ConnectorIO · Feb 29, 2024 · 2acfa02 · 2acfa02
1 parent b7d8d8e
commit 2acfa02
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/bundles/org.connectorio.addons.persistence.memory/pom.xml b/bundles/org.connectorio.addons.persistence.memory/pom.xml
@@ -37,10 +37,6 @@
       <groupId>org.openhab.core.bundles</groupId>
       <artifactId>org.openhab.core.persistence</artifactId>
     </dependency>
-    <dependency>
-      <groupId>org.openhab.core.bundles</groupId>
-      <artifactId>org.openhab.core.config.xml</artifactId>
-    </dependency>
     <dependency>
       <groupId>com.thoughtworks.xstream</groupId>
       <artifactId>xstream</artifactId>

diff --git a/...ain/java/org/connectorio/addons/persistence/memory/internal/MemoryPersistenceService.java b/...ain/java/org/connectorio/addons/persistence/memory/internal/MemoryPersistenceService.java
@@ -55,14 +55,15 @@ public class MemoryPersistenceService implements ModifiablePersistenceService {
 
   // number of entries for each bucket
   private int limit = 100;
-  private String id = "memory";
+  private final String id;
   private String label = "Memory persistence service";
 
   public MemoryPersistenceService(TimeZoneProvider timeZoneProvider) {
     this(ID, timeZoneProvider);
   }
 
   public MemoryPersistenceService(String id, TimeZoneProvider timeZoneProvider) {
+    this.id = id;
     this.timeZoneProvider = timeZoneProvider;
   }
 
@@ -81,9 +82,6 @@ public void update(Map<String, Object> config) {
       }
     }
 
-    if (config.containsKey("id")) {
-      this.id = "" + config.get("id");
-    }
     if (config.containsKey("label")) {
       this.label = "" + config.get("label");
     } else {