]> git.proxmox.com Git - proxmox-widget-toolkit.git/commit
data/*Store: drop storeid requirement
authorDominik Csapak <d.csapak@proxmox.com>
Thu, 11 Mar 2021 09:14:32 +0000 (10:14 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Thu, 11 Mar 2021 12:56:11 +0000 (13:56 +0100)
commit6732337cb95c9c69a08332671bbba3a5d728b724
tree0bd9d1e8b5f7217513472ec28f32113527be6485
parent3f81450686f22e25f779dcc18142086c50e8484d
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>
src/data/ObjectStore.js
src/data/RRDStore.js
src/data/UpdateStore.js