]> git.0d.be Git - empathy.git/commitdiff
account-settings: early return if SASL earlier
authorGuillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Tue, 8 Jan 2013 09:39:11 +0000 (10:39 +0100)
committerGuillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Tue, 8 Jan 2013 09:50:48 +0000 (10:50 +0100)
We should update the URI schemes and service even if the account uses SASL, so
it has to be done *before* the early return.

I did not hit this bug before as the only CM on which we use URI schemes
(rakia) was not implementing SASL channel when I implemented this feature.

https://bugzilla.gnome.org/show_bug.cgi?id=691195

libempathy/empathy-account-settings.c

index 8731479be89302c3cfe602e4ca8589d1df6c6f11..a0e42cc77c06f0ebf197787e0ea708cd0538543f 100644 (file)
@@ -1327,6 +1327,12 @@ empathy_account_settings_account_updated (GObject *source,
       goto out;
     }
 
+  update_account_uri_schemes (settings);
+  update_account_service (settings);
+
+  g_simple_async_result_set_op_res_gboolean (priv->apply_result,
+      g_strv_length (reconnect_required) > 0);
+
   /* Only set the password in the keyring if the CM supports SASL. */
   if (priv->supports_sasl)
     {
@@ -1348,12 +1354,6 @@ empathy_account_settings_account_updated (GObject *source,
       return;
     }
 
-  update_account_uri_schemes (settings);
-  update_account_service (settings);
-
-  g_simple_async_result_set_op_res_gboolean (priv->apply_result,
-      g_strv_length (reconnect_required) > 0);
-
 out:
   empathy_account_settings_discard_changes (settings);