Review Request 129351: Only search for a avatar if loginName isn't empty

[Heiko Becker]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129351/
-----------------------------------------------------------

Review request for KDE Frameworks, David Edmundson and Michael Pyne.


Repository: kcoreaddons


Description
-------

70c745d6147e465ce1e34b6037eb7dedb8e77f27 introduced fetching user
avatars via AccountsService. It's seems pointless to search for an
avatar with an empty user name, e.g. an invalid user, though.
Furthermore it makes kusertest fail with the following message if
ACCOUNTS_SERVICE_ICON_DIR exists:
'Actual (invalidKUser.faceIconPath()): "/var/lib/AccountsService/icons/"
Expected (QString())'


Diffs
-----

  src/lib/util/kuser_unix.cpp d45d0a1ff20b0b93c720d4320b44f4f5e57045a6 

Diff: https://git.reviewboard.kde.org/r/129351/diff/


Testing
-------

kusertest doesn't fail anymore.


Thanks,

Heiko Becker

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161106/74201611/attachment.html>