Review Request: Refactor GroupsModel (from the Contact List) on top of PersonSetModel

George Goldberg grundleborg at googlemail.com
Sun Jul 31 16:53:12 CEST 2011



> On July 31, 2011, 2:50 p.m., David Edmundson wrote:
> > src/ui/grouped-person-set-model.cpp, line 60
> > <http://git.reviewboard.kde.org/r/102156/diff/1/?file=30460#file30460line60>
> >
> >     I think this needs to append the role names from the PersonSetModel too.

PersonSetModelPrivate::setRolesFOrQML() on line 53 should do that.


- George


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


On July 31, 2011, 11:37 a.m., George Goldberg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102156/
> -----------------------------------------------------------
> 
> (Updated July 31, 2011, 11:37 a.m.)
> 
> 
> Review request for Telepathy and Martin Klapetek.
> 
> 
> Summary
> -------
> 
> This patch to telepathy-kde takes the GroupsModel from the contactlist and refactors it into a subclass of PersonSetModel.
> 
> Martin: I'm not entirely convinced I understood your algorithm in GroupsModel for when the groups of a Person change, so I might have ended up making it considerably more inefficient in this review request. Please shout if you see anything that should be changed.
> 
> While writing this summary, I've just realised that I forgot to implement the [Online]UsersCount in this, so assume that I'm aware of that and will add it in a later version of the patch :)
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt b0cf53b6bcf6f2b6395ea24f79f50d6d5344d6fd 
>   src/KTelepathy/GroupedPersonSetModel PRE-CREATION 
>   src/ui/grouped-person-set-model.h PRE-CREATION 
>   src/ui/grouped-person-set-model.cpp PRE-CREATION 
>   src/ui/grouped-person-set-model_p.h PRE-CREATION 
>   src/ui/person-set-model.h 38860ae456a83d698ed19868fbabf4fed6555902 
>   src/ui/person-set-model.cpp 6d01c29f7e7173e2b8af99a709fc3a2137d3115d 
>   src/ui/person-set-model_p.h PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/102156/diff
> 
> 
> Testing
> -------
> 
> PersonSetModel still passes modeltest after the refactoring, and the contactlist behaves as expected when adjusted to use this model instead of the GroupsModel directly.
> 
> 
> Thanks,
> 
> George
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110731/34924b56/attachment.htm 


More information about the KDE-Telepathy mailing list