From: Dominik Csapak <d.csapak@proxmox.com>
Date: Thu, 11 Mar 2021 09:14:32 +0000 (+0100)
Subject: data/*Store: drop storeid requirement
X-Git-Url: https://git.proxmox.com/?p=proxmox-widget-toolkit.git;a=commitdiff_plain;h=6732337cb95c9c69a08332671bbba3a5d728b724

data/*Store: drop storeid requirement

We never use that and it serves no purpose. It probably was meant to
be the upstream config 'storeId' which would add the store to the
Ext.StoreManager. This is unpractical though, since then the store
has to be explicitely destroyed, otherwise the StoreManager retains a
reference and the GC cannot remove the store.

Since donwstream users of the store can simply give the 'storeId'
property anyway if they need to be managed by the StoreManager, drop
the requirement here.

Suggested-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
---

diff --git a/src/data/ObjectStore.js b/src/data/ObjectStore.js
index 860cbfd..cb41f9c 100644
--- a/src/data/ObjectStore.js
+++ b/src/data/ObjectStore.js
@@ -21,10 +21,6 @@ Ext.define('Proxmox.data.ObjectStore', {
 
         config = config || {};
 
-	if (!config.storeid) {
-	    config.storeid = 'proxmox-store-' + ++Ext.idSeed;
-	}
-
         Ext.applyIf(config, {
 	    model: 'KeyValue',
             proxy: {
diff --git a/src/data/RRDStore.js b/src/data/RRDStore.js
index 67ffb57..7c96ded 100644
--- a/src/data/RRDStore.js
+++ b/src/data/RRDStore.js
@@ -37,11 +37,6 @@ Ext.define('Proxmox.data.RRDStore', {
 	    config.interval = 30000;
 	}
 
-	// set a new storeid
-	if (!config.storeid) {
-	    config.storeid = 'rrdstore-' + ++Ext.idSeed;
-	}
-
 	// rrdurl is required
 	if (!config.rrdurl) {
 	    throw "no rrdurl specified";
diff --git a/src/data/UpdateStore.js b/src/data/UpdateStore.js
index be85e4f..cd51089 100644
--- a/src/data/UpdateStore.js
+++ b/src/data/UpdateStore.js
@@ -33,9 +33,6 @@ Ext.define('Proxmox.data.UpdateStore', {
 	    delete config.interval;
 	}
 
-	if (!config.storeid) {
-	    throw "no storeid specified";
-	}
 
 	let load_task = new Ext.util.DelayedTask();