Signed-off-by: Michael Rasmussen <m...@datanom.net> --- www/manager6/Utils.js | 2 +- www/manager6/dc/StorageView.js | 20 ++++++++++---------- www/manager6/lxc/ResourceEdit.js | 16 ++++++++-------- www/manager6/qemu/Clone.js | 2 +- www/manager6/qemu/HDEdit.js | 2 +- www/manager6/qemu/HDMove.js | 2 +- www/manager6/storage/FreeNASEdit.js | 33 ++++++++++++++++++++------------- 7 files changed, 42 insertions(+), 35 deletions(-)
diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js index c6227d0..068c784 100644 --- a/www/manager6/Utils.js +++ b/www/manager6/Utils.js @@ -800,7 +800,7 @@ Ext.define('PVE.Utils', { utilities: { } else if (value === 'drbd') { return 'DRBD'; } else if (value === 'freenas') { - return 'FreeNAS'; + return 'FreeNAS'; } else { return PVE.Utils.unknownText; } diff --git a/www/manager6/dc/StorageView.js b/www/manager6/dc/StorageView.js index c16d464..586d4a9 100644 --- a/www/manager6/dc/StorageView.js +++ b/www/manager6/dc/StorageView.js @@ -59,7 +59,7 @@ Ext.define('PVE.dc.StorageView', { } else if (type === 'zfspool') { editor = 'PVE.storage.ZFSPoolEdit'; } else if (type === 'freenas') { - editor = 'PVE.storage.FreeNASEdit'; + editor = 'PVE.storage.FreeNASEdit'; } else { return; } @@ -194,15 +194,15 @@ Ext.define('PVE.dc.StorageView', { win.show(); } }, - { - text: PVE.Utils.format_storage_type('freenas'), - iconCls: 'fa fa-fw fa-building', - handler: function() { - var win = Ext.create('PVE.storage.FreeNASEdit', {}); - win.on('destroy', reload); - win.show(); - } - } + { + text: PVE.Utils.format_storage_type('freenas'), + iconCls: 'fa fa-fw fa-building', + handler: function() { + var win = Ext.create('PVE.storage.FreeNASEdit', {}); + win.on('destroy', reload); + win.show(); + } + } /* the following type are conidered unstable * so we do not enable that on the GUI for now diff --git a/www/manager6/lxc/ResourceEdit.js b/www/manager6/lxc/ResourceEdit.js index 8bc3e76..4336d86 100644 --- a/www/manager6/lxc/ResourceEdit.js +++ b/www/manager6/lxc/ResourceEdit.js @@ -388,7 +388,7 @@ Ext.define('PVE.lxc.MountPointInputPanel', { var rec = f.store.getById(value); if (rec.data.type === 'zfs' || rec.data.type === 'zfspool' || - rec.data.type === 'freenas') { + rec.data.type === 'freenas') { me.quota.setDisabled(true); me.quota.setValue(false); } else { @@ -403,13 +403,13 @@ Ext.define('PVE.lxc.MountPointInputPanel', { me.hdfilesel.setVisible(true); me.hdsizesel.setDisabled(true); me.hdsizesel.setVisible(false); - } else if (rec.data.type === 'lvm' || - rec.data.type === 'lvmthin' || - rec.data.type === 'rbd' || - rec.data.type === 'sheepdog' || - rec.data.type === 'zfs' || - rec.data.type === 'zfspool' || - rec.data.type === 'freenas') { + } else if (rec.data.type === 'lvm' || + rec.data.type === 'lvmthin' || + rec.data.type === 'rbd' || + rec.data.type === 'sheepdog' || + rec.data.type === 'zfs' || + rec.data.type === 'zfspool' || + rec.data.type === 'freenas') { me.hdfilesel.setDisabled(true); me.hdfilesel.setVisible(false); me.hdsizesel.setDisabled(false); diff --git a/www/manager6/qemu/Clone.js b/www/manager6/qemu/Clone.js index 9de9777..0dbae3b 100644 --- a/www/manager6/qemu/Clone.js +++ b/www/manager6/qemu/Clone.js @@ -117,7 +117,7 @@ Ext.define('PVE.window.Clone', { rec.data.type === 'sheepdog' || rec.data.type === 'zfs' || rec.data.type === 'zfspool' || - rec.data.type === 'freenas' + rec.data.type === 'freenas' ) { formatsel.setValue('raw'); formatsel.setDisabled(true); diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js index 6920c0d..69fc047 100644 --- a/www/manager6/qemu/HDEdit.js +++ b/www/manager6/qemu/HDEdit.js @@ -47,7 +47,7 @@ Ext.define('PVE.qemu.HDInputPanel', { rec.data.type === 'sheepdog' || rec.data.type === 'zfs' || rec.data.type === 'zfspool' || - rec.data.type === 'freenas') { + rec.data.type === 'freenas') { me.hdfilesel.setDisabled(true); me.hdfilesel.setVisible(false); me.formatsel.setValue('raw'); diff --git a/www/manager6/qemu/HDMove.js b/www/manager6/qemu/HDMove.js index 4e69aa7..6caf34b 100644 --- a/www/manager6/qemu/HDMove.js +++ b/www/manager6/qemu/HDMove.js @@ -81,7 +81,7 @@ Ext.define('PVE.window.HDMove', { rec.data.type === 'sheepdog' || rec.data.type === 'zfs' || rec.data.type === 'zfspool' || - rec.data.type === 'freenas' + rec.data.type === 'freenas' ) { me.formatsel.setValue('raw'); me.formatsel.setDisabled(true); diff --git a/www/manager6/storage/FreeNASEdit.js b/www/manager6/storage/FreeNASEdit.js index 5cc9f28..cad51ff 100644 --- a/www/manager6/storage/FreeNASEdit.js +++ b/www/manager6/storage/FreeNASEdit.js @@ -6,7 +6,7 @@ Ext.define('PVE.storage.FreeNASInputPanel', { if (me.isCreate) { values.type = 'freenas'; - values.content = 'images'; + values.content = 'rootdir, images'; } else { delete values.storage; } @@ -14,6 +14,10 @@ Ext.define('PVE.storage.FreeNASInputPanel', { values.disable = values.enable ? 0 : 1; delete values.enable; + if (values.password === '') { + delete values.password; + } + return values; }, @@ -56,7 +60,7 @@ Ext.define('PVE.storage.FreeNASInputPanel', { value: '', fieldLabel: gettext('Initiator Group ID'), allowBlank: false - }, + } ]; me.column2 = [ @@ -67,13 +71,14 @@ Ext.define('PVE.storage.FreeNASInputPanel', { uncheckedValue: 0, fieldLabel: gettext('Enable') }, -/* { - xtype: 'textfield', - name: 'blocksize', - emptyText: '8k', - fieldLabel: gettext('Block Size'), - allowBlank: true - },*/ +/* This setting is first available in the API with next stable release + { + xtype: 'textfield', + name: 'blocksize', + emptyText: '8k', + fieldLabel: gettext('Block Size'), + allowBlank: true + },*/ { xtype: 'textfield', name: 'username', @@ -85,10 +90,10 @@ Ext.define('PVE.storage.FreeNASInputPanel', { xtype: 'textfield', name: 'password', emptyText: '', - inputType: 'password', + inputType: 'password', fieldLabel: gettext('Password'), - allowBlank: false - }, + allowBlank: true + } ]; if (me.isCreate || me.storageId !== 'local') { @@ -129,7 +134,7 @@ Ext.define('PVE.storage.FreeNASEdit', { }); Ext.apply(me, { - subject: 'FreeNAS Storage', + subject: PVE.Utils.format_storage_type('freenas'), isAdd: true, items: [ ipanel ] }); @@ -144,6 +149,8 @@ Ext.define('PVE.storage.FreeNASEdit', { values.nodes = values.nodes.split(','); } values.enable = values.disable ? 0 : 1; + delete values.password; + ipanel.setValues(values); } }); -- 2.1.4 ---- This mail was virus scanned and spam checked before delivery. This mail is also DKIM signed. See header dkim-signature. _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel