| Summary: | Plasmashell logs error to the journal after startup / system boot Binding loop detected for property sourceModel | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | hbr <nepnep> |
| Component: | System Tray widget | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | materka, nate |
| Priority: | NOR | ||
| Version: | 6.3.1 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/-/commit/64e8dfceed4e0038f7af11a2a266dfab10b2a391 | Version Fixed In: | 6.4.0 |
| Sentry Crash Report: | |||
|
Description
hbr
2025-02-19 07:37:18 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5293 Git commit e3f43758f6997dee33e0d4fb7c7afcda44312a35 by Christoph Wolk. Committed on 07/03/2025 at 22:25. Pushed by cwo into branch 'master'. applets/systemtray: remove KSortFilterProxyModel binding loop The systemtray's KSortFilterProxyModels bind to the model exposed on the C++ side, which are in turn QSortFilterProxyModels. This causes a binding loop. The binding is unnecessary in first place though, the (sorted) models are marked as CONSTANT and cannot themselves change (their contents can, of course, but it's still the same sorted models). So we can just set it once on startup and avoid the bindings and any errors or effort associated with them. M +1 -1 applets/systemtray/package/contents/ui/ConfigEntries.qml M +2 -2 applets/systemtray/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-workspace/-/commit/e3f43758f6997dee33e0d4fb7c7afcda44312a35 Git commit 64e8dfceed4e0038f7af11a2a266dfab10b2a391 by Christoph Wolk. Committed on 08/03/2025 at 07:11. Pushed by cwo into branch 'Plasma/6.3'. applets/systemtray: remove KSortFilterProxyModel binding loop The systemtray's KSortFilterProxyModels bind to the model exposed on the C++ side, which are in turn QSortFilterProxyModels. This causes a binding loop. The binding is unnecessary in first place though, the (sorted) models are marked as CONSTANT and cannot themselves change (their contents can, of course, but it's still the same sorted models). So we can just set it once on startup and avoid the bindings and any errors or effort associated with them. (cherry picked from commit e3f43758f6997dee33e0d4fb7c7afcda44312a35) Co-authored-by: Christoph Wolk <cwo.kde@posteo.net> M +1 -1 applets/systemtray/package/contents/ui/ConfigEntries.qml M +2 -2 applets/systemtray/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-workspace/-/commit/64e8dfceed4e0038f7af11a2a266dfab10b2a391 |