WebUI: improve 'set rows' operation performance

PR #23818.

Author: Chocobo1

src/webui/www/private/scripts/dynamicTable.js

@@ -817,18 +817,21 @@ window.qBittorrent.DynamicTable ??= (() => {
          * Replaces all rows with the provided set.
          * More efficient than calling clear() + updateRowData() because existing <tr>
          * elements are reused.
-         * @param {Object[]} rows - Array of row objects, each with a rowId property
+         * @param {Map} rows - A Map with rowId as key
          */
         setRows(rows) {
-            const newRowIds = new Set();
-            for (const row of rows) {
-                this.updateRowData(row);
-                newRowIds.add(`${row.rowId}`);
-            }
-            for (const rowId of this.rows.keys()) {
-                if (!newRowIds.has(rowId))
-                    this.rows.delete(rowId);
+            for (const existingId of this.rows.keys()) {
+                const newVal = rows.get(existingId);
+                if (newVal !== undefined) {
+                    this.updateRowData(newVal);
+                    rows.delete(existingId);
+                }
+                else {
+                    this.rows.delete(existingId);
+                }
             }
+            for (const data of rows.values())
+                this.updateRowData(data);
         }
 
         getTrs() {

src/webui/www/private/scripts/prop-trackers.js

@@ -100,10 +100,11 @@ window.qBittorrent.PropTrackers ??= (() => {
                 const trackers = await response.json();
                 if (trackers) {
                     const notApplicable = "QBT_TR(N/A)QBT_TR[CONTEXT=TrackerListWidget]";
-                    const rows = [];
-                    trackers.each((tracker) => {
-                        rows.push({
-                            rowId: tracker.url,
+                    const rows = new Map();
+                    for (const tracker of trackers) {
+                        const rowId = tracker.url;
+                        rows.set(rowId, {
+                            rowId: rowId,
                             tier: (tracker.tier >= 0) ? tracker.tier : "",
                             btVersion: "",
                             url: tracker.url,
@@ -122,8 +123,9 @@ window.qBittorrent.PropTrackers ??= (() => {
 
                         if (tracker.endpoints !== undefined) {
                             for (const endpoint of tracker.endpoints) {
-                                rows.push({
-                                    rowId: `endpoint|${tracker.url}|${endpoint.name}|${endpoint.bt_version}`,
+                                const rowId = `endpoint|${tracker.url}|${endpoint.name}|${endpoint.bt_version}`;
+                                rows.set(rowId, {
+                                    rowId: rowId,
                                     tier: "",
                                     btVersion: `v${endpoint.bt_version}`,
                                     url: endpoint.name,
@@ -141,7 +143,7 @@ window.qBittorrent.PropTrackers ??= (() => {
                                 });
                             }
                         }
-                    });
+                    }
 
                     torrentTrackersTable.setRows(rows);
                     torrentTrackersTable.updateTable(false);

src/webui/www/private/scripts/prop-webseeds.js

@@ -76,7 +76,14 @@ window.qBittorrent.PropWebseeds ??= (() => {
                 const selectedWebseeds = torrentWebseedsTable.selectedRowsIds();
 
                 const webseeds = await response.json();
-                const rows = (webseeds instanceof Array) ? webseeds.map(ws => ({ rowId: ws.url, url: ws.url })) : [];
+
+                const rows = new Map();
+                if (webseeds instanceof Array) {
+                    for (const seed of webseeds) {
+                        const rowId = seed.url;
+                        rows.set(rowId, { rowId: rowId, url: seed.url });
+                    }
+                }
                 torrentWebseedsTable.setRows(rows);
 
                 torrentWebseedsTable.updateTable(false);