After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 660547 - Get rid of EmpathyContactManager
Get rid of EmpathyContactManager
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: General
2.33.x
Other Linux
: Normal enhancement
: 3.4
Assigned To: empathy-maint
empathy-maint
Depends on: 629517 661489 661805 663327 665038
Blocks:
 
 
Reported: 2011-09-30 08:28 UTC by Guillaume Desmottes
Modified: 2011-12-14 12:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
connection-aggregator: add API to track contacts (3.93 KB, patch)
2011-11-30 12:02 UTC, Guillaume Desmottes
committed Details | Review
event-manager: use the conn aggregator instead of the contact mgr (7.98 KB, patch)
2011-11-30 12:02 UTC, Guillaume Desmottes
committed Details | Review
contact-blocking-dialog: stop using EmpathyContactManager (4.65 KB, patch)
2011-12-02 14:24 UTC, Guillaume Desmottes
committed Details | Review
remove unused empathy_contact_edit_dialog_show() (3.60 KB, patch)
2011-12-14 12:26 UTC, Guillaume Desmottes
committed Details | Review
remove EMPATHY_CONTACT_WIDGET_EDIT_FAVOURITE (5.62 KB, patch)
2011-12-14 12:26 UTC, Guillaume Desmottes
committed Details | Review
main-window: stop instantiating the contact manager (2.10 KB, patch)
2011-12-14 12:27 UTC, Guillaume Desmottes
committed Details | Review
remove useless includes of empathy-contact-manager.h (2.81 KB, patch)
2011-12-14 12:27 UTC, Guillaume Desmottes
committed Details | Review
remove empathy-contact-manager (19.98 KB, patch)
2011-12-14 12:27 UTC, Guillaume Desmottes
committed Details | Review

Description Guillaume Desmottes 2011-09-30 08:28:50 UTC
EmpathyContactManager is still used in a lot of place in Empathy. We should audit the code and remove it as much as possible. Ideally we should use Folks directly. Furthermore, the contact manager uses EmpathyTpContactList which still use the old contact list API.
Comment 1 Guillaume Desmottes 2011-11-30 12:02:17 UTC
Created attachment 202434 [details] [review]
connection-aggregator: add API to track contacts
Comment 2 Guillaume Desmottes 2011-11-30 12:02:19 UTC
Created attachment 202435 [details] [review]
event-manager: use the conn aggregator instead of the contact mgr
Comment 3 Guillaume Desmottes 2011-11-30 12:26:14 UTC
Attachment 202434 [details] pushed as 0fe601d - connection-aggregator: add API to track contacts
Attachment 202435 [details] pushed as 984c03d - event-manager: use the conn aggregator instead of the contact mgr
Comment 4 Guillaume Desmottes 2011-12-02 14:24:11 UTC
Created attachment 202607 [details] [review]
contact-blocking-dialog: stop using EmpathyContactManager
Comment 5 Guillaume Desmottes 2011-12-02 14:30:05 UTC
Attachment 202607 [details] pushed as a9c67f1 - contact-blocking-dialog: stop using EmpathyContactManager
Comment 6 Guillaume Desmottes 2011-12-02 14:30:26 UTC
Bug shouldn't be closed yet.
Comment 7 Guillaume Desmottes 2011-12-14 12:26:55 UTC
Created attachment 203448 [details] [review]
remove unused empathy_contact_edit_dialog_show()
Comment 8 Guillaume Desmottes 2011-12-14 12:26:58 UTC
Created attachment 203449 [details] [review]
remove EMPATHY_CONTACT_WIDGET_EDIT_FAVOURITE

It's not used any more and this code was the last one using the contact
manager.
Comment 9 Guillaume Desmottes 2011-12-14 12:27:02 UTC
Created attachment 203450 [details] [review]
main-window: stop instantiating the contact manager

We don't use it any more.
Comment 10 Guillaume Desmottes 2011-12-14 12:27:04 UTC
Created attachment 203451 [details] [review]
remove useless includes of empathy-contact-manager.h
Comment 11 Guillaume Desmottes 2011-12-14 12:27:07 UTC
Created attachment 203452 [details] [review]
remove empathy-contact-manager
Comment 12 Guillaume Desmottes 2011-12-14 12:30:08 UTC
Attachment 203448 [details] pushed as c7524e2 - remove unused empathy_contact_edit_dialog_show()
Attachment 203449 [details] pushed as 66dbf2d - remove EMPATHY_CONTACT_WIDGET_EDIT_FAVOURITE
Attachment 203450 [details] pushed as 4228684 - main-window: stop instantiating the contact manager
Attachment 203451 [details] pushed as 78120c0 - remove useless includes of empathy-contact-manager.h
Attachment 203452 [details] pushed as 3c136f3 - remove empathy-contact-manager