refact: Improve JS as advised in PR.

This commit is contained in:
Stiliyan Tonev (Bark) 2025-05-24 16:29:55 +03:00
commit f51d07cc94
2 changed files with 7 additions and 14 deletions

View file

@ -265,25 +265,18 @@ window.addEventListener("DOMContentLoaded", (event) => {
} }
}; };
const getFilterSortColumn = (filterName) => {
return LocalPreferences.get(`selected_filter_sort_${filterName}`);
};
const saveFilterSort = (filterName, sort, isReverse) => {
LocalPreferences.set(`selected_filter_sort_${filterName}`, sort);
LocalPreferences.set(`selected_filter_sort_reverse_${filterName}`, isReverse);
};
setStatusFilter = (name) => { setStatusFilter = (name) => {
const currentHash = torrentsTable.getCurrentTorrentID(); const currentHash = torrentsTable.getCurrentTorrentID();
// Save current sorting for this filter. // Save current sorting for this filter.
if (torrentsTable.getSortedColumn()) if (torrentsTable.getSortedColumn()) {
saveFilterSort(selectedStatus, torrentsTable.getSortedColumn(), torrentsTable.reverseSort ?? "0"); LocalPreferences.set(`selected_filter_sort_${selectedStatus}`, torrentsTable.getSortedColumn());
LocalPreferences.set(`selected_filter_sort_reverse_${selectedStatus}`, (torrentsTable.reverseSort ?? "0"));
}
LocalPreferences.set("selected_filter", name); LocalPreferences.set("selected_filter", name);
// If there is a saved sorting column, load it. // If there is a saved sorting column, load it.
const sortColumn = getFilterSortColumn(name); const sortColumn = LocalPreferences.get(`selected_filter_sort_${name}`);
if (sortColumn) { if (sortColumn !== null) {
torrentsTable.setSortedColumn( torrentsTable.setSortedColumn(
sortColumn, sortColumn,
LocalPreferences.get(`selected_filter_sort_reverse_${name}`, "0") LocalPreferences.get(`selected_filter_sort_reverse_${name}`, "0")

View file

@ -681,7 +681,7 @@ window.qBittorrent.DynamicTable ??= (() => {
const oldColumn = this.sortedColumn; const oldColumn = this.sortedColumn;
this.sortedColumn = column; this.sortedColumn = column;
this.reverseSort = reverse ?? "0"; this.reverseSort = reverse ?? "0";
this.setSortedColumnIcon(column, oldColumn, reverse === "1"); this.setSortedColumnIcon(column, oldColumn, (reverse === "1"));
} }
else { else {
// Toggle sort order // Toggle sort order