diff options
| author | Matti Tahvonen <matti.tahvonen@itmill.com> | 2007-05-24 06:57:22 +0000 |
|---|---|---|
| committer | Matti Tahvonen <matti.tahvonen@itmill.com> | 2007-05-24 06:57:22 +0000 |
| commit | 91ac8d001e45b314679c405e83c48104eaab31f5 (patch) | |
| tree | bb40b6433ecb7d9d7418e64067ddc3a1d455bce1 | |
| parent | 0f9fcdb41faa5fbc5c5b985f0fda31a9d813490c (diff) | |
| download | vaadin-framework-91ac8d001e45b314679c405e83c48104eaab31f5.tar.gz vaadin-framework-91ac8d001e45b314679c405e83c48104eaab31f5.zip | |
added check to only remove component if it really is a containers child (avoid errors if programmer makes mistake)
svn changeset:1469/svn branch:trunk
| -rw-r--r-- | src/com/itmill/toolkit/ui/AbstractComponentContainer.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/com/itmill/toolkit/ui/AbstractComponentContainer.java b/src/com/itmill/toolkit/ui/AbstractComponentContainer.java index 60e77dd42f..1719d6aac9 100644 --- a/src/com/itmill/toolkit/ui/AbstractComponentContainer.java +++ b/src/com/itmill/toolkit/ui/AbstractComponentContainer.java @@ -200,7 +200,9 @@ public abstract class AbstractComponentContainer extends AbstractComponent * @see com.itmill.toolkit.ui.ComponentContainer#removeComponent(Component) */ public void removeComponent(Component c) { - c.setParent(null); - fireComponentDetachEvent(c); + if(c.getParent() == this) { + c.setParent(null); + fireComponentDetachEvent(c); + } } } |