From e9ed00f9c22445d2da1180cd38ee13716cb7e707 Mon Sep 17 00:00:00 2001
From: Peter O'Neill <peter.oneill@metaswitch.com>
Date: Thu, 2 May 2013 14:29:12 +0100
Subject: [PATCH] Make sure accounts have at least one attempt to login AFTER
 the reconnect plugin has started listening for RegistrationEvents (so it can
 be notified of failures)

---
 .../sip/communicator/util/account/LoginManager.java  | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/src/net/java/sip/communicator/util/account/LoginManager.java b/src/net/java/sip/communicator/util/account/LoginManager.java
index ab0d7fe90..5e237cc83 100644
--- a/src/net/java/sip/communicator/util/account/LoginManager.java
+++ b/src/net/java/sip/communicator/util/account/LoginManager.java
@@ -125,13 +125,11 @@ private void addAccountsForProtocolProviderFactory(
                 (ProtocolProviderService) UtilActivator.bundleContext
                     .getService(serRef);
 
-            // check whether we have already loaded this provider
-            if(loginRenderer.containsProtocolProviderUI(protocolProvider))
-                continue;
-
-            protocolProvider.addRegistrationStateChangeListener(this);
-
-            loginRenderer.addProtocolProviderUI(protocolProvider);
+            if(!loginRenderer.containsProtocolProviderUI(protocolProvider))
+            {
+                protocolProvider.addRegistrationStateChangeListener(this);
+                loginRenderer.addProtocolProviderUI(protocolProvider);
+            }
 
             Object status =
                 AccountStatusUtils