kotelnik/plasma-applet-thermal-monitor

QML Type Error

aulospl opened this issue · 7 comments

aulospl commented

The app fails to display temperature automatically, but it works when I hit Reload Temperature Sources.
I'm getting this error from journalctl during startup.
Dec 14 12:12:16 caladan plasmashell[693]: file:///home/aulos/.local/share/plasma/plasmoids/org.kde.thermalMonitor/contents/ui/main.qml:312: TypeError: Type error

System info:
Operating System: Antergos Linux
KDE Plasma Version: 5.14.4
Qt Version: 5.12.0
KDE Frameworks Version: 5.53.0
Kernel Version: 4.19.8-arch1-1-ARCH
OS Type: 64-bit
Processors: 4 × Intel® Core™ i5-7200U CPU @ 2.50GHz
Memory: 7.7 GiB of RAM

roachsinai commented

Hi @aulospl could you tell me the command you use to get the error message?

cazacugmihai commented

I have the same error:

file:///home/.../.local/share/plasma/plasmoids/org.kde.thermalMonitor/contents/ui/main.qml:312: TypeError: Type error

Plasma version: 1.2.8

Thank you for this awesome widget!

roachsinai commented

Check PR #54

  • 👍1
cazacugmihai commented

Thank you, @roachsinai!
I confirm that PR #54 it solves the issue.

roachsinai commented

Yeah, thanks for feed back.

Focshole commented

I'm pretty sure i have found the source of the problem. In main.qml:
property var systemmonitorAvailableSources
property var systemmonitorSourcesToAdd
are defined. systemmonitorAvailableSources is being initialized, but systemmonitorAvailableSources isn't. It then reaches the instruction:
systemmonitorAvailableSources.push(source)
so here comes the TypeError. Simply doing:
property var systemmonitorSourcesToAdd: []
fixes the problem. It also adds the missing sources which should load afterwards (no more OFF resources) ;)

roachsinai commented

@Focshole agree with you, exactly what PR #54 did.

  • 👍1