Review Request: Add SSH Contact filtering

Daniele Elmo Domenichelli daniele.domenichelli at gmail.com
Wed Mar 14 01:10:09 UTC 2012



> On March 13, 2012, 11:51 p.m., David Edmundson wrote:
> > KTp/Models/contact-model-item.cpp, line 225
> > <http://git.reviewboard.kde.org/r/104265/diff/1/?file=53055#file53055line225>
> >
> >     Why can't you do it the same way we do audio/video?
> >     
> >     if me->capabilities().streamTubes(QLatin1String("x-ssh-contact")); 
> >     
> >

Because that me->capabilities() returns if you have the capability to handle the channel with requested=false
Since our handler for audio/video is the same for requested=true and requested=false that is not a problem, but conceptually it's wrong and might break if the user changes the handler to something that supports only one direction (quite unlikely though).
But if you check, for krfb a few lines before we are doing the same using KTp::ServiceAvailabilityChecker

We should solve both problems using the desktop files and perhaps subclassing the filter class instead of adding stuff here


> On March 13, 2012, 11:51 p.m., David Edmundson wrote:
> > KTp/Models/accounts-model.h, line 94
> > <http://git.reviewboard.kde.org/r/104265/diff/1/?file=53052#file53052line94>
> >
> >     Long term we can't make a new role for every new stream tube.
> >     
> >     It doesn't make sense to recompile the common library each time someone adds Telepathy support to an app.

I agree, this is a temporary workaroud to the problem and I wasn't sure when I wrote the class if adding the share destop stuff.


- Daniele Elmo


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


On March 13, 2012, 11:46 p.m., Daniele Elmo Domenichelli wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104265/
> -----------------------------------------------------------
> 
> (Updated March 13, 2012, 11:46 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Add "SSH Contact" filtering to the models
> 
> 
> Diffs
> -----
> 
>   KTp/Models/accounts-filter-model.h c23b2843b6ff5a431c4cca275005c1cb4a9ad498 
>   KTp/Models/accounts-filter-model.cpp 5beb201c1a91ef3347f338d7b5301df372833dc5 
>   KTp/Models/accounts-model.h a67912d9d4a727da3556a628295cf2e50f49da1e 
>   KTp/Models/accounts-model.cpp 51fca8802d1522f0b3ed57414c749e8ae0255ece 
>   KTp/Models/contact-model-item.h 4d8ce512de9541c1013aa5cd38859a9bfd45f1f6 
>   KTp/Models/contact-model-item.cpp 50145688be749d0bbb83d814ce99f4f02ec387a8 
> 
> Diff: http://git.reviewboard.kde.org/r/104265/diff/
> 
> 
> Testing
> -------
> 
> Works, but at the moment I don't know how to check if ssh-contact is installed
> 
> 
> Thanks,
> 
> Daniele Elmo Domenichelli
> 
>

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


More information about the KDE-Telepathy mailing list