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>
- if (!config.storeid) {
- config.storeid = 'proxmox-store-' + ++Ext.idSeed;
- }
-
Ext.applyIf(config, {
model: 'KeyValue',
proxy: {
Ext.applyIf(config, {
model: 'KeyValue',
proxy: {
config.interval = 30000;
}
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";
// rrdurl is required
if (!config.rrdurl) {
throw "no rrdurl specified";
delete config.interval;
}
delete config.interval;
}
- if (!config.storeid) {
- throw "no storeid specified";
- }
let load_task = new Ext.util.DelayedTask();
let load_task = new Ext.util.DelayedTask();