Bug 411144

Summary: User Manager - Profile picture is cleared on the kcm page when changing the checkbox states
Product: [Applications] systemsettings Reporter: Alex Nemeth <alex19930329>
Component: kcm_usersAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: bugseforuns, nate
Priority: NOR    
Version First Reported In: 5.16.4   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: video of the problem

Description Alex Nemeth 2019-08-21 17:21:33 UTC
Created attachment 122284 [details]
video of the problem

SUMMARY


STEPS TO REPRODUCE
1. Open the User Manager kcm
2. Assign a profile picture for the user
3. Change the state of the "Log in automatically" checkbox
4. Hit Apply
5. Select a different user on the user list on the left
6. Select the user again

OBSERVED RESULT
Profile picture is set to the default on the kcm page, both on the user list on the left and on the main page.

EXPECTED RESULT
Profile picture is remembered, won't be cleared.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 
KDE Neon unstable
KDE Plasma Version: 5.16.80
KDE Frameworks Version: 5.62.0
Qt Version: 5.12.3
Comment 1 Nate Graham 2019-08-22 19:09:17 UTC
AccountModel::setData() appears to be getting called for every possible change, no matter what is actually changed.
Comment 2 Nate Graham 2019-08-22 19:23:00 UTC
Actually that seems intentional: https://cgit.kde.org/user-manager.git/tree/src/lib/accountmodel.cpp#n246

The problem seems to be that in this case, the face gets cleared rather than re-using the existing data.
Comment 3 Nate Graham 2020-01-21 16:22:34 UTC
Fixed in Plasma 5.18.