this is a wrapper for selecting a storage/disk image it is a simple container with 4 form fields inside which can be reused, whenever we need to select a storage for a disk, image etc.
we have code similar to this four times already (qemu image creation, lxc mp creation, qemu cloning, qemu efidisk creation) so it was time to refactor this and use the storage information from the backend instead of hardcoding values in the frontend for this we need to pass the format=1 parameter to the storage api call, and to not load the fileselector when it is initially disabled Signed-off-by: Dominik Csapak <[email protected]> --- www/manager6/Makefile | 1 + www/manager6/form/DiskStorageSelector.js | 116 +++++++++++++++++++++++++++++++ www/manager6/form/FileSelector.js | 4 +- www/manager6/form/StorageSelector.js | 4 +- 4 files changed, 123 insertions(+), 2 deletions(-) create mode 100644 www/manager6/form/DiskStorageSelector.js diff --git a/www/manager6/Makefile b/www/manager6/Makefile index d39bb49c..52d2f117 100644 --- a/www/manager6/Makefile +++ b/www/manager6/Makefile @@ -52,6 +52,7 @@ JSSRC= \ form/NodeSelector.js \ form/FileSelector.js \ form/StorageSelector.js \ + form/DiskStorageSelector.js \ form/BridgeSelector.js \ form/SecurityGroupSelector.js \ form/IPRefSelector.js \ diff --git a/www/manager6/form/DiskStorageSelector.js b/www/manager6/form/DiskStorageSelector.js new file mode 100644 index 00000000..efb956cd --- /dev/null +++ b/www/manager6/form/DiskStorageSelector.js @@ -0,0 +1,116 @@ +Ext.define('PVE.form.DiskStorageSelector', { + extend: 'Ext.container.Container', + alias: 'widget.pveDiskStorageSelector', + + layout: 'fit', + defaults: { + margin: '0 0 5 0' + }, + + changeStorage: function(f, value) { + var me = this; + var formatsel = me.getComponent('diskformat'); + var hdfilesel = me.getComponent('hdimage'); + var hdsizesel = me.getComponent('disksize'); + + // initial store load, and reset/deletion of the storage + if (!value) { + hdfilesel.setDisabled(true); + hdfilesel.setVisible(false); + + formatsel.setDisabled(true); + return; + } + + var rec = f.store.getById(value); + // if the storage is not defined, or valid, + // we cannot know what to enable/disable + if (!rec) { + return; + } + + var selectformat = false; + if (rec.data.format) { + var format = rec.data.format[0]; // 0 is the formats, 1 the default in the backend + delete format.subvol; // we never need subvol in the gui + selectformat = (Ext.Object.getSize(format) > 1); + } + + var select = !!rec.data.select && !me.hideSelection; + + formatsel.setDisabled(!selectformat); + formatsel.setValue(selectformat ? 'qcow2' : 'raw'); + + hdfilesel.setDisabled(!select); + hdfilesel.setVisible(select); + if (select) { + hdfilesel.setStorage(value); + } + + hdsizesel.setDisabled(select || me.hideSize); + hdsizesel.setVisible(!select && !me.hideSize); + }, + + initComponent: function() { + var me = this; + + me.items = [ + { + xtype: 'pveStorageSelector', + itemId: 'hdstorage', + name: 'hdstorage', + reference: 'hdstorage', + fieldLabel: me.storageLabel || gettext('Storage'), + nodename: me.nodename, + storageContent: me.storageContent, + autoSelect: me.autoSelect, + allowBlank: me.allowBlank, + emptyText: me.emptyText, + listeners: { + change: { + fn: me.changeStorage, + scope: me + } + } + }, + { + xtype: 'pveFileSelector', + name: 'hdimage', + reference: 'hdimage', + itemId: 'hdimage', + fieldLabel: gettext('Disk image'), + nodename: me.nodename, + disabled: me.hideSelection || true, + hidden: me.hideSelection || true + }, + { + xtype: 'numberfield', + itemId: 'disksize', + reference: 'disksize', + name: 'disksize', + fieldLabel: gettext('Disk size') + ' (GB)', + hidden: me.hideSize, + disabled: me.hideSize, + minValue: 0.001, + maxValue: 128*1024, + decimalPrecision: 3, + value: '32', + allowBlank: false + }, + { + xtype: 'pveDiskFormatSelector', + itemId: 'diskformat', + reference: 'diskformat', + name: 'diskformat', + fieldLabel: gettext('Format'), + nodename: me.nodename, + disabled: true, + hidden: me.storageContent === 'rootdir', + value: 'qcow2', + allowBlank: false + } + ]; + + me.callParent(); + } +}); diff --git a/www/manager6/form/FileSelector.js b/www/manager6/form/FileSelector.js index 87a80a23..aaf37d5c 100644 --- a/www/manager6/form/FileSelector.js +++ b/www/manager6/form/FileSelector.js @@ -76,6 +76,8 @@ Ext.define('PVE.form.FileSelector', { me.callParent(); - me.setStorage(me.storage, me.nodename); + if (!me.disabled) { + me.setStorage(me.storage, me.nodename); + } } }); diff --git a/www/manager6/form/StorageSelector.js b/www/manager6/form/StorageSelector.js index 78cd8b3d..184869d1 100644 --- a/www/manager6/form/StorageSelector.js +++ b/www/manager6/form/StorageSelector.js @@ -39,7 +39,9 @@ Ext.define('PVE.form.StorageSelector', { return; } - var params = {}; + var params = { + format: 1 + }; var url = '/api2/json/nodes/' + me.nodename + '/storage'; if (me.storageContent) { params.content = me.storageContent; -- 2.11.0 _______________________________________________ pve-devel mailing list [email protected] https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
