----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122444/ -----------------------------------------------------------
(Updated Feb. 5, 2015, 7:11 p.m.) Review request for Telepathy. Changes ------- Improve the query generation and add a comment Repository: ktp-common-internals Description ------- Currently when account goes online from offline, the contact cache is completely nuked and recreated. This is fast but has one drawback - it will store avatars only of those contacts that are online in the moment of connection, so basically with each connection the available avatars change a bit. To compensate, we've been storing the avatar token in a config file, but that's used only for KTp::Contact. Now with KPeople, if the account is offline, we don't have access to KTp::Contact and have avatars only from the last cache recreation (that means that when you go offline, half of your contacts suddenly loose avatars). This patch now reloads the cache for the account that went offline (second part of this patch is that I've modified the cache to update the database on avatar data change, so for each contact coming online the avatar gets stored in the cache). But as the contact may have not came online during the last connection, it now also loads the avatars from the config file if the database has nothing (this matches the KTp::Contact behavior). Diffs (updated) ----- kpeople/datasourceplugin/CMakeLists.txt 8068977 kpeople/datasourceplugin/im-persons-data-source.cpp f0868b8 Diff: https://git.reviewboard.kde.org/r/122444/diff/ Testing ------- I go offline and all my contacts keep their avatars. Thanks, Martin Klapetek
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
