[Differential] [Request, 39 lines] D4036: When updating another user's avatar don't save icon into our home

David Edmundson noreply at phabricator.kde.org
Mon Jan 9 02:35:49 UTC 2017


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170109/127bcd74/attachment.html>


More information about the Plasma-devel mailing list