[sidebar] Resolve changes in module before switching
ClosedPublic

Authored by davidedmundson on Thu, Feb 13, 7:32 PM.

Details

Summary

After some refactoring we close the module before checking if there are
unsaved changes. We then try to resolve changes on the new KCM that
we've just opened, which never has changes.

BUG: 416834
Fixed-in: 5.18.1

Test Plan

Opened colours in systemsettings
changed it
switched to another kcm

Did the same from within a search

opened kinfocenter
no prompts

Diff Detail

Repository
R124 System Settings
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Thu, Feb 13, 7:32 PM
Restricted Application added a project: Plasma. · View Herald TranscriptThu, Feb 13, 7:32 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Thu, Feb 13, 7:32 PM
bport accepted this revision.Thu, Feb 13, 9:31 PM
bport added a subscriber: bport.

Can reproduce bug and fix work as expected

This revision is now accepted and ready to land.Thu, Feb 13, 9:31 PM
ngraham accepted this revision.Thu, Feb 13, 10:02 PM
ngraham added a subscriber: ngraham.

^^ same

This revision was automatically updated to reflect the committed changes.