squidy/qBittorrent
1
0
Fork 0
mirror of https://github.com/qbittorrent/qBittorrent.git synced 2025-12-31 04:38:04 -06:00
Code Issues Packages Projects Releases Wiki Activity

WebUI: simplify code for filtering nodes

Browse Source 
Now it use object property to store visited information instead of an array.
Also prefix temporary properties with underscore.
There are no user visible changes.

PR #23670.
This commit is contained in:
Chocobo1
2025-12-30 21:50:09 +08:00
committed by GitHub
parent 351b065553
commit 14d2db185a
1 changed files with 9 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
src/webui/www/private/scripts/dynamicTable.js
View File

@@ -2689,32 +2689,30 @@ window.qBittorrent.DynamicTable ??= (() => {
#filterNodes(root, filterTerms) {
const ret = [];
const stack = [root];
const visited = [];
while (stack.length > 0) {
const node = stack.at(-1);
if (node.isFolder && (!this.useVirtualList || !this.isCollapsed(node.rowId))) {
const lastVisited = visited.at(-1);
if ((visited.length <= 0) || (lastVisited !== node)) {
visited.push(node);
if (node._visited === undefined) {
node._visited = true;
stack.push(...node.children);
continue;
}
// has children added or itself matches
if (lastVisited.has_children_added || window.qBittorrent.Misc.containsAllTerms(node.name, filterTerms)) {
// ready to check results from children at this point
if (node._hasChildrenAdded || window.qBittorrent.Misc.containsAllTerms(node.name, filterTerms)) {
ret.push(this.getRow(node));
delete node.has_children_added;
delete node._hasChildrenAdded;
// propagate up
const parent = node.root;
if (parent !== undefined)
parent.has_children_added = true;
parent._hasChildrenAdded = true;
}
visited.pop();
delete node._visited;
}
else {
if (window.qBittorrent.Misc.containsAllTerms(node[this.fileNameColumn], filterTerms)) {
@@ -2722,7 +2720,7 @@ window.qBittorrent.DynamicTable ??= (() => {
const parent = node.root;
if (parent !== undefined)
parent.has_children_added = true;
parent._hasChildrenAdded = true;
}
}