diff options
| author | Juuso Valli <juuso@vaadin.com> | 2014-09-11 16:34:03 +0300 |
|---|---|---|
| committer | Sauli Tähkäpää <sauli@vaadin.com> | 2014-09-15 14:23:54 +0300 |
| commit | 4a60f1a712787cfdadeddef399d177b25806017e (patch) | |
| tree | a96434f9689e12c44d629cd33f7aa7168c5e489c /server/src/com/vaadin/ui/AbstractOrderedLayout.java | |
| parent | 85181e8f2905e3f1174538108fbc3e941946fd56 (diff) | |
| download | vaadin-framework-4a60f1a712787cfdadeddef399d177b25806017e.tar.gz vaadin-framework-4a60f1a712787cfdadeddef399d177b25806017e.zip | |
Make Vaadin component handling proxy-friendly (#14639)
Comparisons with the ==-operator between a proxy and it's underlying
instance fail, so we should use a custom equals method instead.
Change-Id: Iaa86ae830fecbedfb1f55636e25f5affebf5aba3
Diffstat (limited to 'server/src/com/vaadin/ui/AbstractOrderedLayout.java')
| -rw-r--r-- | server/src/com/vaadin/ui/AbstractOrderedLayout.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/server/src/com/vaadin/ui/AbstractOrderedLayout.java b/server/src/com/vaadin/ui/AbstractOrderedLayout.java index 27880db75f..638f6bc3f9 100644 --- a/server/src/com/vaadin/ui/AbstractOrderedLayout.java +++ b/server/src/com/vaadin/ui/AbstractOrderedLayout.java @@ -105,7 +105,7 @@ public abstract class AbstractOrderedLayout extends AbstractLayout implements public void addComponentAsFirst(Component c) { // If c is already in this, we must remove it before proceeding // see ticket #7668 - if (c.getParent() == this) { + if (equals(c.getParent())) { removeComponent(c); } components.addFirst(c); @@ -131,7 +131,7 @@ public abstract class AbstractOrderedLayout extends AbstractLayout implements public void addComponent(Component c, int index) { // If c is already in this, we must remove it before proceeding // see ticket #7668 - if (c.getParent() == this) { + if (equals(c.getParent())) { // When c is removed, all components after it are shifted down if (index > getComponentIndex(c)) { index--; |