Simplify the Nepomuk data extraction Code

Review Request #109481 - Created March 14, 2013 and submitted

Vishesh Handa
    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
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.


  • 1
  • 2
  • 1
  • 4
Description From Last Updated
I think we should just clean the has completely before this step and then discard any checks for it Martin Klapetek Martin Klapetek
Martin Klapetek
Commit Hook
Vishesh Handa
Review request changed

Status: Closed (submitted)