From 2517c76ba8e4ba987bb4efbccf06bbcd972ad034 Mon Sep 17 00:00:00 2001
From: Dominik Csapak <d.csapak@proxmox.com>
Date: Tue, 17 May 2016 11:14:11 +0200
Subject: [PATCH] fix #992: remove network id from gui

since we can simply use the first free one, omit the
selection combobox.

disable the add button if we have 10 or more network
devices

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 www/manager6/lxc/Network.js | 42 +++++++++++++------------------------
 1 file changed, 14 insertions(+), 28 deletions(-)

diff --git a/www/manager6/lxc/Network.js b/www/manager6/lxc/Network.js
index e2ef38b0..7927a8fb 100644
--- a/www/manager6/lxc/Network.js
+++ b/www/manager6/lxc/Network.js
@@ -69,38 +69,22 @@ Ext.define('PVE.lxc.NetworkInputPanel', {
 	    cdata = PVE.Parser.parseLxcNetwork(me.dataCache[me.ifname]);
 	}
 
-	var i, netlist = [];
+	var i;
 	for (i = 0; i < 10; i++) {
-	    netlist.push({ "name": "net" + i.toString() });
-	}
-	
-	var netliststore = Ext.create('Ext.data.Store', {
-	    fields: ['name'],
-	    data: netlist
-	});
-
-	var ifselector = {
-	    xtype: 'combobox',
-	    fieldLabel: gettext('ID'),
-	    store: netliststore,
-	    editable: false,
-	    name: 'id',
-	    value: me.ifname,
-	    disabled: !me.create,
-	    queryMode: 'local',
-	    displayField: 'name',
-	    valueField: 'name',
-	    validator: function(value) {
-		if (me.create && me.dataCache[value]) {
-		    return "Network ID already in use";
-		}
-		// validator can return bool/String
-		/*jslint confusion: true*/
-		return true;
+	    if (me.create && !me.dataCache['net'+i.toString()]) {
+		me.ifname = 'net' + i.toString();
+		break;
 	    }
+	}
+
+	var idselector = {
+	    xtype: 'hidden',
+	    name: 'id',
+	    value: me.ifname
 	};
+
 	me.column1 = [
-	    ifselector,
+	    idselector,
 	    {
 		xtype: 'textfield',
 		name: 'name',
@@ -370,6 +354,7 @@ Ext.define('PVE.lxc.NetworkView', {
 		    records.push(net);
 		});
 		me.store.loadData(records);
+		me.down('button[name=addButton]').setDisabled((records.length >= 10));
 	    }
 	});
     },
@@ -470,6 +455,7 @@ Ext.define('PVE.lxc.NetworkView', {
 	    tbar: [
 		{
 		    text: gettext('Add'),
+		    name: 'addButton',
 		    disabled: !caps.vms['VM.Config.Network'],
 		    handler: function() {
 			var win = Ext.create('PVE.lxc.NetworkEdit', {