From: Artur Signell <artur.signell@itmill.com>
Date: Tue, 16 Mar 2010 12:34:44 +0000 (+0000)
Subject: Fix for #4354 - SplitPanel setComponent methods should check if the component is... 
X-Git-Tag: 6.7.0.beta1~1916
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=de98e505a25b2cae5e2152d540074d60c7bc530a;p=vaadin-framework.git

Fix for #4354 - SplitPanel setComponent methods should check if the component is already set



svn changeset:11908/svn branch:6.3
---

diff --git a/src/com/vaadin/ui/SplitPanel.java b/src/com/vaadin/ui/SplitPanel.java
index 1a149d7394..4319bf7d9c 100644
--- a/src/com/vaadin/ui/SplitPanel.java
+++ b/src/com/vaadin/ui/SplitPanel.java
@@ -105,6 +105,11 @@ public class SplitPanel extends AbstractLayout {
     }
 
     public void setFirstComponent(Component c) {
+        if (firstComponent == c) {
+            // Nothing to do
+            return;
+        }
+
         if (firstComponent != null) {
             // detach old
             removeComponent(firstComponent);
@@ -114,6 +119,11 @@ public class SplitPanel extends AbstractLayout {
     }
 
     public void setSecondComponent(Component c) {
+        if (c == secondComponent) {
+            // Nothing to do
+            return;
+        }
+
         if (secondComponent != null) {
             // detach old
             removeComponent(secondComponent);