From 6c446122402da8f055ee6b6dbc3ad40a49ed4276 Mon Sep 17 00:00:00 2001
From: Lyubomir Marinov <lyubomir.marinov@jitsi.org>
Date: Fri, 4 Mar 2011 17:35:26 +0000
Subject: [PATCH] Fixes issue JITSI-923 IllegalArgumentException: adding
 container's parent to itself at AdvancedConfigurationPanel.showFormContent.

---
 .../advancedconfig/AdvancedConfigurationPanel.java     | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/net/java/sip/communicator/plugin/advancedconfig/AdvancedConfigurationPanel.java b/src/net/java/sip/communicator/plugin/advancedconfig/AdvancedConfigurationPanel.java
index aad5dd9f6..ce88309f5 100644
--- a/src/net/java/sip/communicator/plugin/advancedconfig/AdvancedConfigurationPanel.java
+++ b/src/net/java/sip/communicator/plugin/advancedconfig/AdvancedConfigurationPanel.java
@@ -30,7 +30,8 @@ public class AdvancedConfigurationPanel
                 ListSelectionListener
 {
     /**
-     * The logger.
+     * The <tt>Logger</tt> used by this <tt>AdvancedConfigurationPanel</tt> for
+     * logging output.
      */
     private final Logger logger
         = Logger.getLogger(AdvancedConfigurationPanel.class);
@@ -143,7 +144,11 @@ public void serviceChanged(ServiceEvent event)
 
         ConfigurationForm configForm = (ConfigurationForm) sService;
 
-        if (!configForm.isAdvanced())
+        /*
+         * This AdvancedConfigurationPanel is an advanced ConfigurationForm so
+         * don't try to add it to itself.
+         */
+        if ((configForm == this) || !configForm.isAdvanced())
             return;
 
         switch (event.getType())
@@ -190,6 +195,7 @@ public void addConfigForm(ConfigurationForm configForm)
      *
      * @param isVisible specifies whether the frame is to be visible or not.
      */
+    @Override
     public void setVisible(boolean isVisible)
     {
         if (isVisible && configList.getSelectedIndex() < 0)