From 4fb54d3da0ed9c0b1eafe255df9798f7675866bc Mon Sep 17 00:00:00 2001
From: Tom Piccirello <8296030+Piccirello@users.noreply.github.com>
Date: Sun, 28 Sep 2025 11:19:07 -0700
Subject: [PATCH] WebUI: fix adding torrent when using virtual tables

When using virtual tables we can't rely on fetching the values from the DOM. We should always fetch values directly from the file tree.

PR #23263.
Closes #23241.
---
 src/webui/www/private/addtorrent.html         |  5 +----
 src/webui/www/private/scripts/addtorrent.js   | 19 +++++++++++++------
 src/webui/www/private/scripts/dynamicTable.js |  4 ++++
 3 files changed, 18 insertions(+), 10 deletions(-)

diff --git a/src/webui/www/private/addtorrent.html b/src/webui/www/private/addtorrent.html
index c23b306c5..72a9ace4f 100644
--- a/src/webui/www/private/addtorrent.html
+++ b/src/webui/www/private/addtorrent.html
@@ -85,10 +85,7 @@
             });
 
             window.qBittorrent.pathAutofill.attachPathAutofill();
-            window.qBittorrent.TorrentContent.init("addTorrentFilesTableDiv", window.qBittorrent.DynamicTable.AddTorrentFilesTable);
-
-            if (fetchMetadata)
-                window.qBittorrent.AddTorrent.loadMetadata(source, downloader);
+            window.qBittorrent.AddTorrent.init(source, downloader, fetchMetadata);
         });
     </script>
 
diff --git a/src/webui/www/private/scripts/addtorrent.js b/src/webui/www/private/scripts/addtorrent.js
index 4990eff28..7252a97ae 100644
--- a/src/webui/www/private/scripts/addtorrent.js
+++ b/src/webui/www/private/scripts/addtorrent.js
@@ -29,14 +29,15 @@ window.qBittorrent.AddTorrent ??= (() => {
         return {
             changeCategorySelect: changeCategorySelect,
             changeTMM: changeTMM,
-            loadMetadata: loadMetadata,
             metadataCompleted: metadataCompleted,
             populateMetadata: populateMetadata,
             setWindowId: setWindowId,
-            submitForm: submitForm
+            submitForm: submitForm,
+            init: init
         };
     };
 
+    let table = null;
     let defaultSavePath = "";
     let defaultTempPath = "";
     let defaultTempPathEnabled = false;
@@ -307,10 +308,10 @@ window.qBittorrent.AddTorrent ??= (() => {
         document.getElementById("dlLimitHidden").value = Number(document.getElementById("dlLimitText").value) * 1024;
         document.getElementById("upLimitHidden").value = Number(document.getElementById("upLimitText").value) * 1024;
 
-        document.getElementById("filePriorities").value = [...document.getElementsByClassName("combo_priority")]
-            .filter((el) => !window.qBittorrent.TorrentContent.isFolder(Number(el.dataset.fileId)))
-            .sort((el1, el2) => Number(el1.dataset.fileId) - Number(el2.dataset.fileId))
-            .map((el) => Number(el.value));
+        document.getElementById("filePriorities").value = table.getFileTreeArray()
+            .filter((node) => !node.isFolder)
+            .sort((node1, node2) => (node1.fileId - node2.fileId))
+            .map((node) => node.priority);
 
         if (!isAutoTMMEnabled())
             document.getElementById("useDownloadPathHidden").value = document.getElementById("useDownloadPath").checked;
@@ -326,6 +327,12 @@ window.qBittorrent.AddTorrent ??= (() => {
         }
     };
 
+    const init = (source, downloader, fetchMetadata) => {
+        table = window.qBittorrent.TorrentContent.init("addTorrentFilesTableDiv", window.qBittorrent.DynamicTable.AddTorrentFilesTable);
+        if (fetchMetadata)
+            loadMetadata(source, downloader);
+    };
+
     window.addEventListener("load", async (event) => {
         // user might load this page directly (via browser magnet handler)
         // so wait for crucial initialization to complete
diff --git a/src/webui/www/private/scripts/dynamicTable.js b/src/webui/www/private/scripts/dynamicTable.js
index cb797bb8c..13480ab24 100644
--- a/src/webui/www/private/scripts/dynamicTable.js
+++ b/src/webui/www/private/scripts/dynamicTable.js
@@ -2478,6 +2478,10 @@ window.qBittorrent.DynamicTable ??= (() => {
                 this.#addNodeToTable(child, depth + 1, node);
         }
 
+        getFileTreeArray() {
+            return this.fileTree.toArray();
+        }
+
         getRoot() {
             return this.fileTree.getRoot();
         }