https://bugs.kde.org/show_bug.cgi?id=470784

David Redondo <k...@david-redondo.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/plasma-systemmonitor/-/c
                   |                            |ommit/5f5ae9e1f798df2a44879
                   |                            |59ac2108af0ebaf02d8

--- Comment #6 from David Redondo <k...@david-redondo.de> ---
Git commit 5f5ae9e1f798df2a4487959ac2108af0ebaf02d8 by David Redondo.
Committed on 29/08/2023 at 14:32.
Pushed by davidre into branch 'master'.

Provide a default for enabled columns

Due to a race in item instantiation if the process table view
is instantiated before the column configuration dialog the enabled
attributes will be initially empty. However we still enable columns
which then will be filtered out, confusing the proxy model stack
as there will be no valid parents anymore because the first column
has been filtered out but still claiming to have rows.
Delaying assignment of attributes causes a stack overflow due
to QSortFilterProxyModelPrivate::recursiveChildAcceptsRows calling
itself.
Providing here a default enabled column is a straightfoward fix
instead of trying to find the exact point of failure in the the proxymodels
provided by us and upstream and matches the default for columnDisplay.

M  +1    -1    src/table/ColumnConfigurationDialog.qml

https://invent.kde.org/plasma/plasma-systemmonitor/-/commit/5f5ae9e1f798df2a4487959ac2108af0ebaf02d8

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to