Currently in log_store_empathy_search_hit_new() log-store-empathy get a list of valid accounts from the AM, because of the way Empathy handled it. In TPL it shoudln't be used only valid accounts, or the search will cut off all the accounts that for example were removed but still in the TPL log history. Probably calling tp_account_manager_get_valid_accounts() is useles and since it needs preparation, it is actually not needed. Instead a query to retrieve all the current logged account would be its replacement. From TPL LogStore's point of view, it doesn't matter if an account is valid or not, just if it has some LogEntry to show or not.
A branch fixing it, removing AccountManager references from TplLogStoreXML (former TplLogStoreEmpathy) is at http://git.collabora.co.uk/?p=user/kalfa/telepathy-logger.git;a=shortlog;h=refs/heads/log-store-AM This branch uses a patch (tpl_get_tp_account()) which has been introduces by another branch (danni's) still to be reviewed (not yet to review)
(In reply to comment #1) > A branch fixing it, removing AccountManager references from TplLogStoreXML > (former TplLogStoreEmpathy) is at > http://git.collabora.co.uk/?p=user/kalfa/telepathy-logger.git;a=shortlog;h=refs/heads/log-store-AM > > This branch uses a patch (tpl_get_tp_account()) which has been introduces by > another branch (danni's) still to be reviewed Reference ?
This blocks Empathy showing logs for removed accounts: https://bugzilla.gnome.org/show_bug.cgi?id=610455
-- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/telepathy/telepathy-logger/issues/2.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.