Review Request 111256: Import the KPeople KTP integration plugins

[Martin Klapetek]

> On June 26, 2013, 9:16 p.m., David Edmundson wrote:
> > kpeople/common/impersonsdatasource.cpp, line 213
> > <http://git.reviewboard.kde.org/r/111256/diff/2/?file=165907#file165907line213>
> >
> >     Turns out this isn't ideal, if I do "group by accounts" whilst one of my accounts is offline, everything freezes for quite some time.
> >     
> >     I have a cunning plan on how to fix this:
> >      - we create a new PersonsModelFeature that includes the IM accountID as a new model role, using most of this query here. This will be in KTp code only, not in libkpeople as it is very app-specific
> >     
> >      - We then have an accountID which we can use in KTpTranslationProxyModel for KTp::AccountRole.
> >     
> >     Not too important to discuss here.
> >     
> >

Yeah, especially because it's a sync query. +1000 on KTp specific feature.


> On June 26, 2013, 9:16 p.m., David Edmundson wrote:
> > kpeople/common/impersonsdatasource.cpp, line 242
> > <http://git.reviewboard.kde.org/r/111256/diff/2/?file=165907#file165907line242>
> >
> >     FWIW, I think the reason we do objectPath is so we could do
> >     
> >     Tp::AccountPtr = Tp::Account::create(objectPath);
> >     
> >     rather than going through the AM.

I don't think you can do just that, it would miss all the AM features, no? That's why it goes through it.


- Martin


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


On June 26, 2013, 2:33 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111256/
> -----------------------------------------------------------
> 
> (Updated June 26, 2013, 2:33 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Import the KPeople KTP integration plugins
> 
> There are two plugins, one for the datasource which provides
> IM data in real time.
> 
> The other is an actions plugin which provides actions that can be started on a PersonsData.
> 
> We need to provide KTp integration into libkpeople in a plugin, but we also want to access this
> singleton of accounts/contacts from the other plugin and from the KTp Models. To do this a static library
> ktpkpeople is created and everything links against that.
> 
> KTp Models the actions plugin and the im data source all link against this.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt a83069d59fed945d04560662e3a307ef33b28332 
>   KTp/Models/CMakeLists.txt 720a5b02465b825b08d12d7c43342ad1ba69c9d1 
>   kpeople/actionsplugin/CMakeLists.txt PRE-CREATION 
>   kpeople/actionsplugin/implugin.h PRE-CREATION 
>   kpeople/actionsplugin/implugin.cpp PRE-CREATION 
>   kpeople/actionsplugin/ktp_kpeople_plugin.desktop PRE-CREATION 
>   kpeople/common/CMakeLists.txt PRE-CREATION 
>   kpeople/common/impersonsdatasource.h PRE-CREATION 
>   kpeople/common/impersonsdatasource.cpp PRE-CREATION 
>   kpeople/datasourceplugin/CMakeLists.txt PRE-CREATION 
>   kpeople/datasourceplugin/im_persons_data_source_plugin.desktop PRE-CREATION 
>   kpeople/datasourceplugin/impersonsdatasourceplugin.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/111256/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130626/133c2eb9/attachment.html>