Bug 497773

Summary:	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
Product:	[Plasma] plasmashell	Reporter:	pallaswept <pallaswept>
Component:	System Monitor widgets	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	CONFIRMED ---
Severity:	normal	CC:	ahiemstra, john.kizer, nate, notmart
Priority:	NOR
Version:	6.2.4
Target Milestone:	1.0
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

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 :)