davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY We want to save the face using AccountsService, but for backwards compatibility we also save the icon into ~/.face for old apps/DisplayManagers that still expect that. This works when setting a face as the current user, but doesn't make sense when setting the icon of another user. We can't fix it to write into another user's home, so instead of trying just skip that step and only update AccountsService. BUG: 365662 BUG: 360005 BUG: 371663 TEST PLAN Changed my user's avatar. AccountsService updated ~/.face updated Changed another user's avatar from my account. Accounts service updated, ~/.face was unchanged REPOSITORY R128 User Manager BRANCH master REVISION DETAIL https://phabricator.kde.org/D4036 AFFECTED FILES src/accountinfo.cpp EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: davidedmundson, #plasma Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, andreaska, sebas