[Kimchi-devel] [PATCH] [Kimchi] Issue #818: Frontend doesn't update the PCI tab after a PCI attachment

[sguimaraes943 at gmail.com]

From: Samuel Guimarães <sguimaraes943 at gmail.com>

Signed-off-by: Samuel Guimarães <sguimaraes943 at gmail.com>
---
 ui/js/src/kimchi.guest_edit_main.js | 40 ++++++++++++++++++++-----------------
 1 file changed, 22 insertions(+), 18 deletions(-)

diff --git a/ui/js/src/kimchi.guest_edit_main.js b/ui/js/src/kimchi.guest_edit_main.js
index b51f97f..40d8189 100644
--- a/ui/js/src/kimchi.guest_edit_main.js
+++ b/ui/js/src/kimchi.guest_edit_main.js
@@ -427,8 +427,7 @@ kimchi.guest_edit_main = function() {
         });
     };
 
-    var filterPCINodes = function(group, text, targetName) {
-        targetName = targetName || null;
+    var filterPCINodes = function(group, text) {
         text = text.trim().split(" ");
         var rows = $('.body', '#form-guest-edit-pci').find('div');
         if(text === ""){
@@ -462,11 +461,11 @@ kimchi.guest_edit_main = function() {
             });
         });
         $('select', '#form-guest-edit-pci').change(function() {
-            filterPCINodes($(this).val(), $('input#guest-edit-pci-filter', '#form-guest-edit-pci').val(), '');
+            filterPCINodes($(this).val(), $('input#guest-edit-pci-filter', '#form-guest-edit-pci').val());
         });
         $('select', '#form-guest-edit-pci').selectpicker();
         $('input#guest-edit-pci-filter', '#form-guest-edit-pci').on('keyup', function() {
-            filterPCINodes($('select', '#form-guest-edit-pci').val(), $(this).val(), '');
+            filterPCINodes($('select', '#form-guest-edit-pci').val(), $(this).val());
         });
     };
     var setupNode = function(arrPCIDevices, iconClass) {
@@ -502,31 +501,36 @@ kimchi.guest_edit_main = function() {
                 var id = obj.parent().parent().attr('id');
                 if (objIcon.hasClass('fa-ban')) {
                     kimchi.removeVMPCIDevice(kimchi.selectedGuest, id, function() {
-                        kimchi.getAvailableHostPCIDevices(function(arrPCIDevices1) {
-                            kimchi.getVMPCIDevices(kimchi.selectedGuest, function(vmPCIs1) {
-                                for (var k = 0; k < arrPCIDevices1.length; k++) {
-                                    $('button i', '#' + arrPCIDevices1[k].name).removeClass('fa-ban').addClass('fa-power-off');
+                        kimchi.getAvailableHostPCIDevices(function(arrPCIDevices) {
+                            kimchi.getVMPCIDevices(kimchi.selectedGuest, function(vmPCIs) {
+                                for (var k = 0; k < arrPCIDevices.length; k++) {
+                                    $('#' + arrPCIDevices[k].name + '.item').removeClass('enabled').addClass('disabled');
+                                    $('#' + arrPCIDevices[k].name + ' .action-area button i').removeClass('fa-ban').addClass('fa-power-off');
                                 }
-                                for (var k = 0; k < vmPCIs1.length; k++) {
-                                    $('button i', '#' + arrPCIDevices1[k].name).removeClass('fa-power-off').addClass('fa-ban');
+                                for (var k = 0; k < vmPCIs.length; k++) {
+                                    $('#' + arrPCIDevices[k].name + '.item').removeClass('disabled').addClass('enabled');
+                                    $('#' + arrPCIDevices[k].name + ' .action-area button i').removeClass('fa-power-off').addClass('fa-ban');
                                 }
                             });
                         });
                         //id is for the object that is being added back to the available PCI devices
-                        filterPCINodes($('select', '#form-guest-edit-pci').val(), $('input#guest-edit-pci-filter', '#form-guest-edit-pci').val(), id);
-                    });
+                        filterPCINodes($('select', '#form-guest-edit-pci').val(), $('input#guest-edit-pci-filter', '#form-guest-edit-pci').val());
+                    }, null);
                 } else {
                     kimchi.addVMPCIDevice(kimchi.selectedGuest, {
                         name: id
                     }, function() {
-                        kimchi.getVMPCIDevices(kimchi.selectedGuest, function(vmPCIs1) {
-                            for (var k = 0; k < vmPCIs1.length; k++) {
-                                $('button i', '#' + arrPCIDevices1[k].name).removeClass('fa-power-off').addClass('fa-ban');
-                            }
+                        kimchi.getAvailableHostPCIDevices(function(arrPCIDevices) {
+                            kimchi.getVMPCIDevices(kimchi.selectedGuest, function(vmPCIs) {
+                                for (var k = 0; k < vmPCIs.length; k++) {
+                                    $('#' + vmPCIs[k].name + '.item').removeClass('disabled').addClass('enabled');
+                                    $('#' + vmPCIs[k].name + ' .action-area button i').removeClass('fa-power-off').addClass('fa-ban');
+                                }
+                            });
                         });
                         //id is for the object that is being removed from the available PCI devices
-                        filterPCINodes($('select', '#form-guest-edit-pci').val(), $('input#guest-edit-pci-filter', '#form-guest-edit-pci').val(), id);
-                    });
+                        filterPCINodes($('select', '#form-guest-edit-pci').val(), $('input#guest-edit-pci-filter', '#form-guest-edit-pci').val());
+                    }, null);
                 }
             });
             kimchi.getPCIDeviceCompanions(pciDeviceName, function(infoData) {
-- 
1.9.3