fix #4370: ui: fix regression of preselecting tree element on login

We want to select a tree element based on the fragment in the url after
a login. For which we must wait for the tree (ResourceStore) to load.

Since we recently introduced the loading of '/cluster/options' to
load the overrides of tags, etc. we fire a 'load' event with the
intention of triggering a rerender of the tree (to include the new
colors). If that triggers before the ResourceStore is loaded,
we try to set the element once (which fails) and nothing is selected.

To fix this, we only trigger it if it was already loaded and is not
currently loading. Otherwise it will be either redrawn after the ongoing
load, or by the first load.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js
index 4dd3cb93ee1dced90f5f71b7da373d966dec1432..8c118fa25e0b40b2142f2a2002d8d4aefdd5d462 100644 (file)
--- a/www/manager6/Utils.js
+++ b/www/manager6/Utils.js
@@ -1918,7 +1918,10 @@ Ext.define('PVE.Utils', {
        }
 
        Ext.ComponentQuery.query('pveResourceTree')[0].setUserCls(`proxmox-tags-${shape}`);
-       PVE.data.ResourceStore.fireEvent('load');
+
+       if (!PVE.data.ResourceStore.isLoading() && PVE.data.ResourceStore.isLoaded()) {
+           PVE.data.ResourceStore.fireEvent('load');
+       }
        Ext.GlobalEvents.fireEvent('loadedUiOptions');
     },