Review Request 111228: Switch ContactsModel between legacy mode and KPeople powered

Aleix Pol Gonzalez aleixpol at kde.org
Tue Jun 25 10:46:39 UTC 2013



> On June 25, 2013, 9:48 a.m., Aleix Pol Gonzalez wrote:
> > KTp/Models/contacts-model.cpp, line 88
> > <http://git.reviewboard.kde.org/r/111228/diff/1/?file=165661#file165661line88>
> >
> >     Don't you have to set the account manager for the ContactListModel?
> 
> Martin Klapetek wrote:
>     That is what's happening - there's #ifndef
> 
> David Edmundson wrote:
>     and assuming the question also meant 
>     "do you have to set it for the PersonsModel". No, it has it's own, because we didn't want normal KPeople users to have to worry about setting that (which makes sense I think)
>     
>     We do need to figure out what we're doing with the account managers. Having two factories is very inefficient (you double all DBus traffic), something needs changing, not sure what.

Uh, my bad. Thanks Martin! >.<

Maybe it's possible to re-use the one in KPeople from the ktp-contactlist?

Either way, please ship it.

/me is excited about finally getting to use it! :D


- Aleix


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111228/#review35033
-----------------------------------------------------------


On June 25, 2013, 1:30 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111228/
> -----------------------------------------------------------
> 
> (Updated June 25, 2013, 1:30 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Switch ContactsModel between legacy mode and KPeople powered
> 
> Include a proxy model that translates between KPeople and KTp roles so that it can seemlessly support the two
> 
> 
> --
> 
> It includes copied ktptranslationproxymodel, but renamed as it doesn't make sense to call it the KTp translation inside KTp.
> 
> Code is pushed to kpeople branch.
> 
> I want Martin to cherry-pick (w. review) useful commits on the contact-list from his branch into master till things vaguely work. 
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt a83069d59fed945d04560662e3a307ef33b28332 
>   KTp/Models/CMakeLists.txt 720a5b02465b825b08d12d7c43342ad1ba69c9d1 
>   KTp/Models/contacts-model.cpp 9506151f5489d9a3f80f184834da26cb90141f3c 
>   KTp/Models/kpeopletranslationproxy.h PRE-CREATION 
>   KTp/Models/kpeopletranslationproxy.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/111228/diff/
> 
> 
> Testing
> -------
> 
> Opened regular ktp-contact-list.
> 
> It was trying to show things from PersonModel. It was a bit broken, unsurprisingly, as the old delegates don't support Person rows. 
> 
> For some reason presence icons and avatars don't appear. This implies the translation proxy must be broken, as if we have to change the delegates, then it won't work with the legacy model. Filteringrtin
> 
> Tried running contact list plasmoid, (as that should 'just work') however we no longer have anything exposing rolenames so it doesn't work. Needs exposing into some way that can be shared. (moving setRoleNames from ContactsListModel to ContactsModel should suffice)
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130625/b46183ca/attachment-0001.html>


More information about the KDE-Telepathy mailing list