Bug 471153

Summary: Night Color KCM resets "switching times" to "Always Off" on open
Product: [Applications] systemsettings Reporter: Patrick Silva <bugseforuns>
Component: kcm_nightcolorAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: bednarczyk.pawel, bharadwaj.raju777, isma.af, kwin-bugs-null, nate, nicolas.fella, qydwhotmail
Priority: NOR Keywords: qt6
Version First Reported In: master   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/plasma-workspace/-/commit/d61b7eb4af3ba1092c106eac9df10f3e62aa1e1b Version Fixed/Implemented In: 6
Sentry Crash Report:

Description Patrick Silva 2023-06-17 13:16:07 UTC 
STEPS TO REPRODUCE
1. open Night Color KCM
2. close System Settings without changing anything
3. 

OBSERVED RESULT
the save/discard prompt shows up

EXPECTED RESULT
System Settings closes

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.27.80
KDE Frameworks Version: 5.240.0
Qt Version: 6.5.1
Graphics Platform: Wayland
Comment 1 Nate Graham 2023-06-21 04:49:20 UTC 
Can confirm. The problem appears to be that the "Switching times" combobox gets reset at launch, changing its value and causing the KCM to enter a dirty state immediately.
Comment 2 Bharadwaj Raju 2023-06-23 11:41:59 UTC 
Looks like the currentIndexChanged() on the combobox gets called too early
Comment 3 Bharadwaj Raju 2023-06-23 14:07:10 UTC 
For some reason, this problem doesn't appear in kcmshell6. Only in systemsettings.

When running under systemsettings, the onActiveChanged connection on kcm.nightColorSettings never gets called, while it does happen under kcmshell6. I have no idea why (yet).
Comment 4 Bharadwaj Raju 2023-07-07 09:44:47 UTC 
I have a hack solution for this, but I don't feel good about ignoring such a perplexing underlying problem. Do you think it's worth making an MR for the hack?
Comment 5 Nate Graham 2023-07-07 13:42:07 UTC 
Depending on how difficult it would be to fix in a way you consider more correct, it might be worth submitting your hack patch as an RFC to stimulate discussion around the subject.
Comment 6 Bug Janitor Service 2023-07-07 14:08:32 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3050
Comment 7 Fushan Wen 2023-09-30 09:13:08 UTC 
(In reply to Bharadwaj Raju from comment #3)
> For some reason, this problem doesn't appear in kcmshell6. Only in
> systemsettings.
> 
> When running under systemsettings, the onActiveChanged connection on
> kcm.nightColorSettings never gets called, while it does happen under
> kcmshell6. I have no idea why (yet).

This looks like https://invent.kde.org/plasma/systemsettings/-/merge_requests/258

The signal/slot is broken in systemsettings
Comment 8 Patrick Silva 2023-11-13 11:41:23 UTC 
*** Bug 476927 has been marked as a duplicate of this bug. ***