Bug 488558

Summary: System monitor applets are empty with Qt 6.8
Product: [Plasma] plasmashell Reporter: Antonio Rojas <arojas>
Component: System Monitor widgetsAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED UPSTREAM    
Severity: normal CC: ahiemstra, fanzhuyifan, ferlaxzhu, idoybh2, kde, nate, notmart, p.r.worrall, thederpyworld
Priority: HI Keywords: regression
Version First Reported In: git-stable-Plasma/6.1   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
URL: https://bugreports.qt.io/browse/QTBUG-126398
Latest Commit: Version Fixed In: Qt 6.8.0
Sentry Crash Report:

Description Antonio Rojas 2024-06-15 21:59:36 UTC 
System monitor applets don't work with Qt 6.8. In the panel they are invisible, the popup shows the background lines but nothing else. Console output shows:

file:///usr/share/ksysguard/sensorfaces/org.kde.ksysguard.linechart/contents/ui/LineChart.qml:53:9: Unable to assign QJsonArray to QStringList

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Arch Linux x86_64
KDE Plasma Version: 6.0.90
KDE Frameworks Version: 6.3.0
Qt Version: 6.8 beta 1
Comment 1 Paul Worrall 2024-06-16 07:21:46 UTC 
In the System Monitor app the sensors with text output are working OK, it's the graphs that aren't working.

Operating System: Arch Linux 
KDE Plasma Version: 6.0.90
KDE Frameworks Version: 6.3.0
Qt Version: 6.8.0
KDE Gear 24.05.1
Graphics Platform: Wayland
Comment 2 Paul Worrall 2024-06-16 08:12:33 UTC 
*** Bug 488568 has been marked as a duplicate of this bug. ***
Comment 3 Antonio Rojas 2024-06-16 11:11:06 UTC 
QJsonArray is now interpreted as a Sequence https://code.qt.io/cgit/qt/qtdeclarative.git/commit?h=6.8&id=8dbcea319a20b0bc905d2988dc9f35394f73c2dc

Explicitly casting to Array fixes the issue, but there are a lot if instances to fix:

--- a/faces/facepackages/linechart/contents/ui/LineChart.qml
+++ b/faces/facepackages/linechart/contents/ui/LineChart.qml
@@ -50,7 +50,7 @@ Charts.LineChart {
 
     Sensors.SensorDataModel {
         id: sensorsModel
-        sensors: chart.controller.highPrioritySensorIds
+        sensors: Array.from(chart.controller.highPrioritySensorIds)
         updateRateLimit: chart.controller.updateRateLimit
         sensorLabels: chart.controller.sensorLabels
Comment 4 David Redondo 2024-06-17 07:39:58 UTC 
Great not even a Changelog category.
Comment 5 Nate Graham 2024-07-19 14:45:41 UTC 
https://bugreports.qt.io/browse/QTBUG-126398 is fixed now in a later snapshot of Q 6.8; closing.
Comment 6 Antonio Rojas 2024-07-25 13:52:25 UTC 
*** Bug 490811 has been marked as a duplicate of this bug. ***