Review Request 109481: Simplify the Nepomuk data extraction Code

[Commit Hook]

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


This review has been submitted with commit 52840e528720590414f2d617565db51c3325fe76 by Vishesh Handa to branch master.

- Commit Hook


On March 14, 2013, 1:30 p.m., Vishesh Handa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109481/
> -----------------------------------------------------------
> 
> (Updated March 14, 2013, 1:30 p.m.)
> 
> 
> Review request for Telepathy and Martin Klapetek.
> 
> 
> Description
> -------
> 
>     Simplify the Nepomuk data extraction Code
>     
>     Instead of using the Nepomuk property URIs as keys in ContactItem we now
>     use the roles instead. This simplifies the data extraction part from the
>     model as we can keep a direct mapping in the query (Binding Name => Role)
>     
>     Additionally, the ContactItem class could also load the data on its own
>     by using the Nepomuk Resource class. It now uses the same query as the
>     PersonsModel and fetches all the data in one go. This is much faster and
>     simpler. Though, it results in a little bit of code duplication.
>     
>     And finally, the ResourceWatcher now reloads the contacts which are
>     modified instead of just changing those specific properties. This is
>     done because in many cases such as - ImAccounts, Emails, Photos, and
>     ContactGroups, one needs to query the Nepomuk DB in order to fetch more
>     information.
> 
> 
> Diffs
> -----
> 
>   src/contact-item.h e970a6d 
>   src/contact-item.cpp e6b8277 
>   src/person-item.cpp 7a8f520 
>   src/persons-model.cpp 4e82688 
>   src/resource-watcher-service.h 8484162 
>   src/resource-watcher-service.cpp 6aafb83 
>   src/tests/duplicatestest.cpp 3ebde5a 
> 
> Diff: http://git.reviewboard.kde.org/r/109481/diff/
> 
> 
> Testing
> -------
> 
> Tested by using the ktp-contactlist. It looks the same in both the cases. 
> 
> Also tested the ResourceWatcher by -
> * manually changing an imNickName of one contact and it got updated.
> * Adding a nco:photo for a nco:PersonContact and it also got updated.
> 
> 
> Thanks,
> 
> Vishesh Handa
> 
>

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