Proxmox/pve-manager
5
0
Fork 0
Code Pull Requests Releases Activity

ui: vm/usbedit: don't use getUSBValue to avoid possible duplicate "usb3"

Browse Source 
As getUSBValue() always added the "usb3" property if the selected
device was recognized as USB3. But we can now also pass such a device
as USB2 so lift this restriction - which also fixes a bug for USB3
selected devices with the USB3 checkbox set.

The solution also avoids the me.down query, which is nice too.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2019-09-23 17:06:21 +02:00
parent 700cd718ec
commit 3db3e3eccc
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
www/manager6/qemu/USBEdit.js
View File

@ -55,10 +55,12 @@ Ext.define('PVE.qemu.USBInputPanel', {
var type = me.down('radiofield').getGroupValue();
switch (type) {
case 'spice':
val = 'spice'; break;
val = 'spice';
break;
case 'hostdevice':
case 'port':
val = me.down('pveUSBSelector[name=' + type + ']').getUSBValue();
val = 'host=' + values[type];
delete values[type];
break;
default:
throw "invalid type selected";
@ -102,7 +104,6 @@ Ext.define('PVE.qemu.USBInputPanel', {
allowBlank: false,
fieldLabel: 'Choose Device',
labelAlign: 'right',
submitValue: false
},
{
name: 'usb',
@ -122,7 +123,6 @@ Ext.define('PVE.qemu.USBInputPanel', {
allowBlank: false,
fieldLabel: gettext('Choose Port'),
labelAlign: 'right',
submitValue: false
},
{
xtype: 'checkbox',