From 1ea54b58b49d47c2de7ea0d491eb103c08edfd01 Mon Sep 17 00:00:00 2001
From: George Politis <666f6f@java.net>
Date: Sun, 20 Sep 2009 15:19:12 +0000
Subject: [PATCH] Fixed: If the OTR button can receives a null MetaContact (for
 example when the current chat is in a chat room) the OTR button stays
 enabled, even changes when I click on it (reported by Lubomir).

---
 .../plugin/otr/OtrMetaContactButton.java      | 43 ++++++++-----------
 1 file changed, 18 insertions(+), 25 deletions(-)

diff --git a/src/net/java/sip/communicator/plugin/otr/OtrMetaContactButton.java b/src/net/java/sip/communicator/plugin/otr/OtrMetaContactButton.java
index 341740c46..18b4ec5d2 100644
--- a/src/net/java/sip/communicator/plugin/otr/OtrMetaContactButton.java
+++ b/src/net/java/sip/communicator/plugin/otr/OtrMetaContactButton.java
@@ -139,47 +139,42 @@ public String getName()
 
     public void setCurrentContact(Contact contact)
     {
-        if (this.contact != contact)
-        {
-            this.contact = contact;
+        if (this.contact == contact) 
+            return;
 
+        this.contact = contact;
+        if (contact != null)
+        {
             this.setStatus(OtrActivator.scOtrEngine.getSessionStatus(contact));
             this.setPolicy(OtrActivator.scOtrEngine.getContactPolicy(contact));
         }
+        else
+        {
+            this.setStatus(SessionStatus.PLAINTEXT);
+            this.setPolicy(null);
+        }
     }
 
     public void setCurrentContact(MetaContact metaContact)
     {
-        /*
-         * TODO What if metaContact is null? Does it mean that this.contact
-         * should become null?
-         */
-        if (metaContact != null)
-        {
-            Contact defaultContact = metaContact.getDefaultContact();
-
-            if (defaultContact != null)
-                setCurrentContact(defaultContact);
-        }
+        setCurrentContact((metaContact == null) ? null : metaContact
+            .getDefaultContact());
     }
 
     private void setPolicy(OtrPolicy contactPolicy)
     {
-        getButton().setEnabled(contactPolicy.getEnableManual());
+        getButton().setEnabled(
+            contactPolicy != null && contactPolicy.getEnableManual());
     }
 
     private void setStatus(SessionStatus status)
     {
-        if (contact == null)
-            return;
-
         String urlKey;
         switch (status)
         {
         case ENCRYPTED:
-            urlKey
-                = OtrActivator.scOtrKeyManager.isVerified(contact)
-                    ? "plugin.otr.ENCRYPTED_ICON_22x22"
+            urlKey =
+                OtrActivator.scOtrKeyManager.isVerified(contact) ? "plugin.otr.ENCRYPTED_ICON_22x22"
                     : "plugin.otr.ENCRYPTED_UNVERIFIED_ICON_22x22";
             break;
         case FINISHED:
@@ -194,10 +189,8 @@ private void setStatus(SessionStatus status)
 
         try
         {
-            getButton()
-                .setImage(
-                    ImageIO.read(
-                        OtrActivator.resourceService.getImageURL(urlKey)));
+            getButton().setImage(
+                ImageIO.read(OtrActivator.resourceService.getImageURL(urlKey)));
         }
         catch (IOException e)
         {