ui: esxi storage edit: make user and password always editable

Those can change somewhat frequently, e.g., as part of credentials
rotation policies.

Keep the 'server' one for now as is, while it can make sense to change
that too, e.g., from IP to FQDN, it's currently uncommon for PVE to
allow that in the UI, if we should evaluate all storage types for such
things and for that we also should improve the re-mount/activate story
in the storage lib.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

diff --git a/www/manager6/storage/ESXIEdit.js b/www/manager6/storage/ESXIEdit.js
index 01c8bd237aa6e2be98bf83ae58427e43cc0dd9f9..9ffff32ae437e9a3f71b8b0f688907a7d343ff27 100644 (file)
--- a/www/manager6/storage/ESXIEdit.js
+++ b/www/manager6/storage/ESXIEdit.js
@@ -20,30 +20,26 @@ Ext.define('PVE.storage.ESXIInputPanel', {
        me.column1 = [
            {
                xtype: 'pmxDisplayEditField',
-               editable: me.isCreate,
                name: 'server',
                fieldLabel: gettext('Server'),
+               editable: me.isCreate,
+               emptyText: gettext('IP address or hostname'),
                allowBlank: false,
            },
            {
-               xtype: 'pmxDisplayEditField',
-               editable: me.isCreate,
+               xtype: 'textfield',
                name: 'username',
                fieldLabel: gettext('Username'),
                allowBlank: false,
            },
            {
-               xtype: 'pmxDisplayEditField',
-               editable: me.isCreate,
+               xtype: 'proxmoxtextfield',
                name: 'password',
-               value: me.isCreate ? '' : '********',
-               minLength: 1,
-               editConfig: {
-                   inputType: 'password',
-                   name: 'password',
-               },
                fieldLabel: gettext('Password'),
-               allowBlank: false,
+               inputType: 'password',
+               emptyText: gettext('Unchanged'),
+               minLength: 1,
+               allowBlank: !me.isCreate,
            },
        ];