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 671035 - Accounts dialog doesn't update when account goes online
Accounts dialog doesn't update when account goes online
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Accounts
3.3.x
Other Linux
: Normal normal
: 3.4
Assigned To: empathy-maint
empathy-maint
Depends on:
Blocks:
 
 
Reported: 2012-02-29 09:08 UTC by Vincent Untz
Modified: 2012-03-15 15:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
factor out account_is_selected() (2.49 KB, patch)
2012-03-15 15:04 UTC, Guillaume Desmottes
committed Details | Review
create_dialog_content: check if the TpConnection is still valid (1.04 KB, patch)
2012-03-15 15:05 UTC, Guillaume Desmottes
committed Details | Review
reload the account widget when connection is (dis)connected (2.38 KB, patch)
2012-03-15 15:05 UTC, Guillaume Desmottes
committed Details | Review

Description Vincent Untz 2012-02-29 09:08:51 UTC
If an account is offline, and I got to it in the account dialog, I get a "Go online to edit your personal information." hint.

Now, if I go online for this account... then the message is still there.

Moving to another account and back to the first one, I can see my personal information.

(In the same way: if I'm online with this dialog opened and go offline, I can still see the personal information, instead of having the "Can't edit" hint).
Comment 1 Guillaume Desmottes 2012-03-15 15:04:59 UTC
Created attachment 209843 [details] [review]
factor out account_is_selected()
Comment 2 Guillaume Desmottes 2012-03-15 15:05:01 UTC
Created attachment 209844 [details] [review]
create_dialog_content: check if the TpConnection is still valid

tp_connection_get_contacts_by_handle() will raise a warning if the
TpConnection has been invalidated.
Comment 3 Guillaume Desmottes 2012-03-15 15:05:04 UTC
Created attachment 209845 [details] [review]
reload the account widget when connection is (dis)connected
Comment 4 Xavier Claessens 2012-03-15 15:12:06 UTC
+1 but I think it's a bug of TpAccount that it gives invalidated connection. IMO TpAccount should connect "invalidated" signal on its TpConnection and set it to NULL.
Comment 5 Guillaume Desmottes 2012-03-15 15:14:48 UTC
Attachment 209843 [details] pushed as 500ef85 - factor out account_is_selected()
Attachment 209844 [details] pushed as a3c10ee - create_dialog_content: check if the TpConnection is still valid
Attachment 209845 [details] pushed as f17b4a2 - reload the account widget when connection is (dis)connected
Comment 6 Guillaume Desmottes 2012-03-15 15:30:48 UTC
(In reply to comment #4)
> +1 but I think it's a bug of TpAccount that it gives invalidated connection.
> IMO TpAccount should connect "invalidated" signal on its TpConnection and set
> it to NULL.

I opened https://bugs.freedesktop.org/show_bug.cgi?id=47364