D9951: [KUserProxy] Adjust to accounts service

Aleix Pol Gonzalez noreply at phabricator.kde.org
Thu Jan 18 03:27:39 UTC 2018


apol added a comment.


  LGTM

INLINE COMMENTS

> kuserproxy.cpp:66
> +    if (path == m_user.faceIconPath() || path == QDir::homePath() + QLatin1String("/.face.icon")
> +            || path == accountsServiceIconPath + QLatin1String("/") + m_user.loginName()) {
>          // we need to force updates, even when the path doesn't change,

QLatin1Char('/')

REPOSITORY
  R296 KDeclarative

REVISION DETAIL
  https://phabricator.kde.org/D9951

To: broulik, #plasma
Cc: apol, plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180118/35c73b9e/attachment.html>


More information about the Kde-frameworks-devel mailing list