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

Remove unused noScsi parameter

Browse Source 
SCSI is a valid bus for Disks and CD drives, so it should always
be appear in the bus list.

Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Emmanuel Kasper 2017-10-02 10:37:31 +02:00 committed by Wolfgang Bumiller
parent 7206af4b04
commit f5aea3f018
2 changed files with 2 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
www/manager6/form/BusTypeSelector.js
View File

@ -4,8 +4,6 @@ Ext.define('PVE.form.BusTypeSelector', {
noVirtIO: false,
noScsi: false,
initComponent: function() {
var me = this;
@ -15,9 +13,7 @@ Ext.define('PVE.form.BusTypeSelector', {
me.comboItems.push(['virtio', 'VirtIO']);
}
if (!me.noScsi) {
me.comboItems.push(['scsi', 'SCSI']);
}
me.comboItems.push(['scsi', 'SCSI']);
me.callParent();
}

6
www/manager6/form/ControllerSelector.js
View File

@ -13,8 +13,6 @@ Ext.define('PVE.form.ControllerSelector', {
noVirtIO: false,
noScsi: false,
vmconfig: {}, // used to check for existing devices
sortByPreviousUsage: function(vmconfig, controllerList) {
@ -75,8 +73,7 @@ Ext.define('PVE.form.ControllerSelector', {
Ext.Array.each(clist, function(controller) {
var confid, i;
if ((controller === 'virtio' && me.noVirtIO) ||
(controller === 'scsi' && me.noScsi)) {
if (controller === 'virtio' && me.noVirtIO) {
return; //continue
}
me.down('field[name=controller]').setValue(controller);
@ -106,7 +103,6 @@ Ext.define('PVE.form.ControllerSelector', {
name: 'controller',
value: PVE.qemu.OSDefaults.generic.busType,
noVirtIO: me.noVirtIO,
noScsi: me.noScsi,
allowBlank: false,
flex: 2,
listeners: {