Bug 497773 - System Monitor Sensor widget presets don't permit some title characters, but the "Save Settings as Preset" button gives no feedback whether it succeeded or failed
Summary: System Monitor Sensor widget presets don't permit some title characters, but ...
Status: CONFIRMED
Alias: None
Product: plasmashell
Classification: Plasma
Component: System Monitor widgets (show other bugs)
Version: 6.2.4
Platform: openSUSE Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-12-22 07:50 UTC by pallaswept
Modified: 2025-01-04 04:19 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description pallaswept 2024-12-22 07:50:00 UTC
SUMMARY
Saving System Monitor settings as preset fails with errors in the journal.

STEPS TO REPRODUCE
1. Customize System Monitor widget
2. Attempt to save settings as preset

OBSERVED RESULT
Nothing happens at all.

Journalctl shows:

Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml:18:1: QML ConfigSensors: Created graphical object was not placed in the graphics scene.
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_chartFace
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_chartFaceDefault
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_sensorColors
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_sensorColorsDefault
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_sensorIds
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_sensorIdsDefault
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_textOnlySensorIds
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_textOnlySensorIdsDefault
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_title
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_titleDefault
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_totalSensor
Dec 22 17:38:38 Pallas plasmashell[2884]: file:///usr/share/plasma/plasmoids/org.kde.plasma.systemmonitor/contents/ui/config/ConfigSensors.qml: Setting initial properties failed: ConfigSensors does not have a property called cfg_totalSensorDefault


EXPECTED RESULT
A cool preset

SOFTWARE/OS VERSIONS

KDE Plasma Version: 6.2.4
KDE Frameworks Version: 6.9.0
Qt Version: 6.8.1

ADDITIONAL INFORMATION

Pretty sure this is different to 485164?
Comment 1 John Kizer 2024-12-30 06:38:45 UTC
Hi - I'm on the same KDE/Qt versions, I don't see any visual feedback when clicking Save Settings as Preset, and I get errors in the journal as well. However, I do also get a new preset, with the currently saved "Title" on the Appearance tab, listed at the bottom of the "Load Preset" list.

Could you please double-check that no new record, with the Title you set, was created at the bottom of the "Load Preset" list?
Comment 2 pallaswept 2024-12-30 06:42:45 UTC
(In reply to John Kizer from comment #1)
> However, I do also get a new preset, with the currently saved "Title" on the
> Appearance tab, listed at the bottom of the "Load Preset" list.
> 
> Could you please double-check that no new record, with the Title you set,
> was created at the bottom of the "Load Preset" list?

That's interesting. Confirmed that I do not have a new entry in that list.
Comment 3 pallaswept 2024-12-30 06:44:15 UTC
Aha. My preset was named "CPU / RAM". If I remove the slash, I get a new entry as you described!
Comment 4 John Kizer 2024-12-30 06:52:01 UTC
Ah, good catch! I can reproduce that - updating the title, thanks!
Comment 5 pallaswept 2024-12-30 06:58:40 UTC
(In reply to John Kizer from comment #4)
> Ah, good catch! I can reproduce that - updating the title, thanks!

Thanks to you! Telling me that the name would appear in that list gave me the hint.

I'll just drop the special characters from my title for now, conveniently I don't display it anyway, so that's a good workaround for me. This is obviously in good hands now :)