proxmox-mirrors/pve-manager
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-manager synced 2025-07-24 16:59:02 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

ui: fire config change: always pass required parameters

Browse Source 
As otherwise the listeners might choke on the missing records.

For our usage it was only a problem when one changed the tree UI
options while having the Datacenter -> Summary panel open, as that
listens to the UI changes to adapt the shown gauges for e.g., memory,
storage, or CPUs, and thus actually uses the current result (unlike
we do in the store load listener).

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2023-03-10 15:38:17 +01:00
parent e98edb4c37
commit 4aa70a6b3a
2 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
www/manager6/UIOptions.js
View File

@ -99,9 +99,7 @@ Ext.define('PVE.UIOptions', {
},
fireUIConfigChanged: function() {
if (!PVE.data.ResourceStore.isLoading() && PVE.data.ResourceStore.isLoaded()) {
PVE.data.ResourceStore.fireEvent('load');
}
PVE.data.ResourceStore.refresh();
Ext.GlobalEvents.fireEvent('loadedUiOptions');
},
});

9
www/manager6/data/ResourceStore.js
View File

@ -53,6 +53,15 @@ Ext.define('PVE.data.ResourceStore', {
return '';
},
refresh: function() {
let me = this;
// can only refresh if we're loaded at least once and are not currently loading
if (!me.isLoading() && me.isLoaded()) {
let records = (me.getData().getSource() || me.getData()).getRange()
me.fireEvent('load', me, records);
}
},
constructor: function(config) {
let me = this;