diff options
| author | Artur Signell <artur.signell@itmill.com> | 2008-12-08 13:43:03 +0000 |
|---|---|---|
| committer | Artur Signell <artur.signell@itmill.com> | 2008-12-08 13:43:03 +0000 |
| commit | 6ca7337c6d09f03e54f090030f25d9ec1ffdf6ba (patch) | |
| tree | 61161e5c1109cb61827b3ea971450f28f357ea2b /src/com/itmill/toolkit/ui/AbstractOrderedLayout.java | |
| parent | d6d3c709063147bf1cd46d7de5bee94ee3ee8472 (diff) | |
| download | vaadin-framework-6ca7337c6d09f03e54f090030f25d9ec1ffdf6ba.tar.gz vaadin-framework-6ca7337c6d09f03e54f090030f25d9ec1ffdf6ba.zip | |
Fix for #2294 - Sanity check for setComponentAlignment
svn changeset:6120/svn branch:trunk
Diffstat (limited to 'src/com/itmill/toolkit/ui/AbstractOrderedLayout.java')
| -rw-r--r-- | src/com/itmill/toolkit/ui/AbstractOrderedLayout.java | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/src/com/itmill/toolkit/ui/AbstractOrderedLayout.java b/src/com/itmill/toolkit/ui/AbstractOrderedLayout.java index cb3d277222..fe9d897d71 100644 --- a/src/com/itmill/toolkit/ui/AbstractOrderedLayout.java +++ b/src/com/itmill/toolkit/ui/AbstractOrderedLayout.java @@ -235,9 +235,14 @@ public abstract class AbstractOrderedLayout extends AbstractLayout implements */ public void setComponentAlignment(Component childComponent, int horizontalAlignment, int verticalAlignment) { - componentToAlignment.put(childComponent, new Integer( - horizontalAlignment + verticalAlignment)); - requestRepaint(); + if (components.contains(childComponent)) { + componentToAlignment.put(childComponent, new Integer( + horizontalAlignment + verticalAlignment)); + requestRepaint(); + } else { + throw new IllegalArgumentException( + "Component must be added to layout before using setComponentAlignment()"); + } } /* |