proxmox-mirrors/pve-manager
1
0
Fork 0
mirror of https://git.proxmox.com/git/pve-manager synced 2025-07-27 15:21:28 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

give error instead of disabling efidisk button

Browse Source 
when seabios is selected, we do not want to simply
disable the add efi button, because it is not clear
why it is disabled

so we give a meaningful error message instead

the button is still 'just' disabled when there
is already an efidisk, because then it is clear
that you alread have one

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2016-09-13 15:12:26 +02:00 committed by Dietmar Maurer
parent f3777880e9
commit 1535df627b
1 changed files with 18 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
www/manager6/qemu/HardwareView.js
View File

@ -436,12 +436,20 @@ Ext.define('PVE.qemu.HardwareView', {
iconCls: 'pve-itype-icon-storage', iconCls: 'pve-itype-icon-storage',
disabled: !caps.vms['VM.Config.Disk'], disabled: !caps.vms['VM.Config.Disk'],
handler: function() { handler: function() {
var win = Ext.create('PVE.qemu.EFIDiskEdit', {
url: '/api2/extjs/' + baseurl, var rstoredata = me.rstore.getData().map;
pveSelNode: me.pveSelNode // check if ovmf is configured
}); if (rstoredata.bios && rstoredata.bios.data.value === 'ovmf') {
win.on('destroy', reload); var win = Ext.create('PVE.qemu.EFIDiskEdit', {
win.show(); url: '/api2/extjs/' + baseurl,
pveSelNode: me.pveSelNode
});
win.on('destroy', reload);
win.show();
} else {
Ext.Msg.alert('Error',gettext('Please select OVMF(UEFI) as BIOS first.'));
}
} }
}); });
@ -449,16 +457,10 @@ Ext.define('PVE.qemu.HardwareView', {
var sm = me.getSelectionModel(); var sm = me.getSelectionModel();
var rec = sm.getSelection()[0]; var rec = sm.getSelection()[0];
// check if there is already an efidisk // disable button when we have an efidisk already
var rstoredata = me.rstore.getData().map; // disable is ok in this case, because you can instantly
if (rstoredata.bios && // see that there is already one
rstoredata.bios.data.value === 'ovmf' && efidisk_menuitem.setDisabled(me.rstore.getData().map.efidisk0 !== undefined);
!rstoredata.efidisk0) {
// we have ovmf configured and no efidisk
efidisk_menuitem.setDisabled(false);
} else {
efidisk_menuitem.setDisabled(true);
}
if (!rec) { if (!rec) {
remove_btn.disable(); remove_btn.disable();