From bb284008cc421dc563f4b1d0c5d1de4b0159f4d5 Mon Sep 17 00:00:00 2001
From: RubenPX <rubenmarchante@gmail.com>
Date: Thu, 28 Apr 2022 15:01:39 +0200
Subject: [PATCH] Fix re render table

---
 ereuse_devicehub/static/js/main_inventory.js | 22 +++++++++-----------
 1 file changed, 10 insertions(+), 12 deletions(-)

diff --git a/ereuse_devicehub/static/js/main_inventory.js b/ereuse_devicehub/static/js/main_inventory.js
index c230363d..6e0c9d12 100644
--- a/ereuse_devicehub/static/js/main_inventory.js
+++ b/ereuse_devicehub/static/js/main_inventory.js
@@ -299,15 +299,13 @@ async function processSelectedDevices() {
             const tmpDiv = document.createElement("div")
             tmpDiv.innerHTML = newRequest
 
-            const oldTable = Array.from(document.querySelectorAll("table.table > tbody > tr .deviceSelect")).map(x => x.attributes["data-device-dhid"].value)
             const newTable = Array.from(tmpDiv.querySelectorAll("table.table > tbody > tr .deviceSelect")).map(x => x.attributes["data-device-dhid"].value)
 
-            for (let i = 0; i < oldTable.length; i++) {
-                if (!newTable.includes(oldTable[i])) {
-                    // variable from device_list.html --> See: ereuse_devicehub\templates\inventory\device_list.html (Ln: 411)
-                    table.rows().remove(i)
+            table.rows().dt.activeRows.forEach(row => {
+                if (!newTable.includes(row.querySelector("input").attributes["data-device-dhid"].value)) {
+                    row.remove()
                 }
-            }
+            })
         }
     }
 
@@ -352,12 +350,12 @@ async function processSelectedDevices() {
 
     const listHTML = $("#LotsSelector")
 
-    // Get selected devices
-    const selectedDevicesIDs = $.map($(".deviceSelect").filter(":checked"), (x) => parseInt($(x).attr("data")));
-    if (selectedDevicesIDs.length <= 0) {
-        listHTML.html("<li style=\"color: red; text-align: center\">No devices selected</li>");
-        return;
-    }
+     // Get selected devices
+     const selectedDevicesIDs = $.map($(".deviceSelect").filter(":checked"), (x) => parseInt($(x).attr("data")));
+     if (selectedDevicesIDs.length <= 0) {
+         listHTML.html("<li style=\"color: red; text-align: center\">No devices selected</li>");
+         return;
+     }
 
     // Initialize Actions list, and set checkbox triggers
     const actions = new Actions();