Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QML Type Error #52

Open
aulospl opened this issue Dec 14, 2018 · 7 comments
Open

QML Type Error #52

aulospl opened this issue Dec 14, 2018 · 7 comments

Comments

@aulospl
Copy link

aulospl commented Dec 14, 2018

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
Copy link

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

@cazacugmihai
Copy link

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
Copy link

Check PR #54

@cazacugmihai
Copy link

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

@roachsinai
Copy link

Yeah, thanks for feed back.

@Focshole
Copy link

Focshole commented Oct 23, 2019

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
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants