[Kimchi-devel] [PATCH][Kimchi] Issue: #1008 Issues while editing a VEPA network

Ramon Medeiros ramonn at linux.vnet.ibm.com
Thu Oct 6 17:36:44 UTC 2016


The javascript was iterating a dict without any propose, and copying the
same objects inside of it. Just remove the iteration and adequate the
correct vars.

Signed-off-by: Ramon Medeiros <ramonn at linux.vnet.ibm.com>
---
 ui/js/src/kimchi.network_edit_main.js | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/ui/js/src/kimchi.network_edit_main.js b/ui/js/src/kimchi.network_edit_main.js
index 4dd830a..ab1e285 100644
--- a/ui/js/src/kimchi.network_edit_main.js
+++ b/ui/js/src/kimchi.network_edit_main.js
@@ -93,16 +93,10 @@ kimchi.setupNetworkFormEventForEdit = function(network) {
 
     var loadIfaces = function(interfaceFilterArray){
         var buildInterfaceOpts = function(result) {
-            var currentIfaces = network['interfaces'];
-            for (var i = 0; i < currentIfaces.length; i++) {
-                kimchi.getInterface(currentIfaces[i], function(iface) {
-                    result.push(iface);
-                } , null, true);
-            }
             kimchi.createInterfacesOpts(result, interfaceFilterArray);
 
-            for (var i = 0; i < currentIfaces.length; i++) {
-                $("#networkDestinationID option[value='" + currentIfaces[i] + "']").attr('selected','selected');
+            for (var i = 0; i < result.length; i++) {
+                $("#networkDestinationID option[value='" + result[i]["name"] + "']").attr('selected','selected');
             }
             $('#networkDestinationID').selectpicker('refresh');
         };
-- 
2.7.4




More information about the Kimchi-devel mailing list