Fixes some IlligalStateExceptions and NPE when using icq protocol.

src/net/java/sip/communicator/impl/gui/main/chat/toolBars/MainToolBar.java

@@ -335,6 +335,10 @@ public void chatChanged(ChatPanel chatPanel)
                 while(contacts.hasNext())
                 {
                     Contact c = contacts.next();
+                    
+                    if(!c.getProtocolProvider().isRegistered())
+                        continue;
+                    
                     OperationSetServerStoredContactInfo infoOpSet =
                         c.getProtocolProvider().getOperationSet(
                             OperationSetServerStoredContactInfo.class);

src/net/java/sip/communicator/impl/gui/main/contactlist/ContactListTreeCellRenderer.java

@@ -753,6 +753,10 @@ private void initButtonsPanel(UIContact uiContact)
             while(contacts.hasNext() && !hasPhone)
             {
                 Contact contact = contacts.next();
+
+                if(!contact.getProtocolProvider().isRegistered())
+                    continue;
+
                 OperationSetServerStoredContactInfo infoOpSet =
                     contact.getProtocolProvider().getOperationSet(
                         OperationSetServerStoredContactInfo.class);

src/net/java/sip/communicator/impl/gui/main/contactlist/contactsource/MetaUIContact.java

@@ -423,7 +423,8 @@ private void loadTooltip(final ExtendedTooltip tip)
                 protocolContact.getProtocolProvider().getOperationSet(
                     OperationSetServerStoredContactInfo.class);
 
-            if(infoOpSet != null)
+            if(infoOpSet != null
+                && protocolContact.getProtocolProvider().isRegistered())
             {
                 Iterator<GenericDetail> details =
                     infoOpSet.requestAllDetailsForContact(protocolContact,

src/net/java/sip/communicator/impl/protocol/icq/InfoRetreiver.java

@@ -154,9 +154,9 @@ protected List<GenericDetail> retrieveDetails(String uin)
             result = new LinkedList<GenericDetail>();
 
         // put even empty result to bypass further retrieve
-        retreivedDetails.put(uin, responseRetriever.result);
+        retreivedDetails.put(uin, result);
 
-        return responseRetriever.result;
+        return result;
     }
 
     /**