https://bugs.kde.org/show_bug.cgi?id=445544
David Redondo <k...@david-redondo.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |https://invent.kde.org/plas
| |ma/plasma-systemmonitor/com
| |mit/a96bc99a3650c3bf2e48ab1
| |287dc779dbba87c51
Resolution|--- |FIXED
Status|ASSIGNED |RESOLVED
Version Fixed In| |5.23.4
--- Comment #2 from David Redondo <k...@david-redondo.de> ---
Git commit a96bc99a3650c3bf2e48ab1287dc779dbba87c51 by David Redondo.
Committed on 02/12/2021 at 14:51.
Pushed by davidre into branch 'master'.
ApplicationsTable: Remove unneeded ProcessSortFilterModel
ProcessSortFilterModel handles filtering pids and to which user a process
belongs over a normal QSortFilterProxyModel. In doing so it also
sets the filterKeyColumn to nameColumn in findColumns(). Note that the name
of the name columns of the process and application table is different
("name" and "appName"), so the filterKeyColumn is also set by
ApplicationTableView.
However this is racey depending on when the binding is evaluated.
Since we don't need the extra features of the ProcessSortFilterModel we can use
the existing SortFilterProxyModel in charge of filtering columns (something
that
the ProcessSortFilterModel also did) to also handle the row filtering.
FIXED-IN:5.23.4
M +4 -11
src/faces/applicationstable/contents/ui/ApplicationsTableView.qml
https://invent.kde.org/plasma/plasma-systemmonitor/commit/a96bc99a3650c3bf2e48ab1287dc779dbba87c51
--
You are receiving this mail because:
You are watching all bug changes.
