Fixes issue JITSI-923 IllegalArgumentException: adding container's parent to itself at AdvancedConfigurationPanel.showFormContent.

cusax-fix
Lyubomir Marinov 15 years ago
parent e38663e9cb
commit 6c44612240

@ -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)

Loading…
Cancel
Save