ui: storage selector: only check for nodename before loading storage

but allow an empty nodename to be set. This is in preparation for
allowing a cluster view as well, where an empty node name will
indicate that no node is selected.

Since the reloadStorageList function still returns early if there is
no nodename, this should not lead to any issues.

Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>

diff --git a/www/manager6/form/StorageSelector.js b/www/manager6/form/StorageSelector.js
index 631eb3781ceb5d7c11314e212573c2a128c78a9c..bb56f1598acd9c6633118a739da3dc6a8bb9450b 100644 (file)
--- a/www/manager6/form/StorageSelector.js
+++ b/www/manager6/form/StorageSelector.js
@@ -74,7 +74,9 @@ Ext.define('PVE.form.StorageSelector', {
     setNodename: function(nodename) {
        var me = this;
 
-       if (!nodename || me.nodename === nodename) {
+       nodename = nodename || '';
+
+       if (me.nodename === nodename) {
            return;
        }
 
@@ -103,9 +105,7 @@ Ext.define('PVE.form.StorageSelector', {
 
        me.callParent();
 
-       if (nodename) {
-           me.setNodename(nodename);
-       }
+       me.setNodename(nodename);
     },
 }, function() {
     Ext.define('pve-storage-status', {