[Bug 31195] Implement models

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Oct 28 15:57:06 CEST 2010


https://bugs.freedesktop.org/show_bug.cgi?id=31195

--- Comment #2 from Paolo Capriotti <paolo.capriotti at collabora.co.uk> 2010-10-28 06:57:05 PDT ---
Thanks for the detailed review. The code you are referring to is located here:

http://git.collabora.co.uk/?p=user/paolo/telepathy-qt4.git;a=summary

The latest developments are in the 'contacts' branch. As you can see, I removed
all calls to becomeReady and other telepathy initialization functions. The
models now assume that the objects passed to it are already initialized.

To avoid duplicating a lot of the bookkeeping of the accounts model, I also
merged contacts and accounts in a single hierarchical model (where accounts are
at level 1, and contacts for each account at level 2).

You can also use this model if you only need the accounts, and a flag could be
added to save the overhead involved in maintaining contact data for this use
case.

If you only need an aggregated contact list, you can still use this model, and
wrap it in a FlatModelProxy, which will flatten it out removing first level
nodes.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.



More information about the telepathy-bugs mailing list