diff options
| author | Artur Signell <artur.signell@itmill.com> | 2011-12-08 13:04:33 +0000 |
|---|---|---|
| committer | Artur Signell <artur.signell@itmill.com> | 2011-12-08 13:04:33 +0000 |
| commit | e968734dfa87136d7a488156789edf77cac1d7f6 (patch) | |
| tree | 8207ac2e178dbf06ce388aa14ac966442fbfdde2 /src/com/vaadin/data/util/AbstractBeanContainer.java | |
| parent | 946ea244388af573fb8cb7b7d0523046b7ceb863 (diff) | |
| download | vaadin-framework-e968734dfa87136d7a488156789edf77cac1d7f6.tar.gz vaadin-framework-e968734dfa87136d7a488156789edf77cac1d7f6.zip | |
#6603 Added IllegalArgumentException to javadoc and added test
svn changeset:22320/svn branch:6.7
Diffstat (limited to 'src/com/vaadin/data/util/AbstractBeanContainer.java')
| -rw-r--r-- | src/com/vaadin/data/util/AbstractBeanContainer.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/com/vaadin/data/util/AbstractBeanContainer.java b/src/com/vaadin/data/util/AbstractBeanContainer.java index 0e4feaab32..6260e05518 100644 --- a/src/com/vaadin/data/util/AbstractBeanContainer.java +++ b/src/com/vaadin/data/util/AbstractBeanContainer.java @@ -626,9 +626,12 @@ public abstract class AbstractBeanContainer<IDTYPE, BEANTYPE> extends * The collection of beans to add. Must not be null. * @throws IllegalStateException * if no bean identifier resolver has been set + * @throws IllegalArgumentException + * if the resolver returns a null itemId for one of the beans in + * the collection */ protected void addAll(Collection<? extends BEANTYPE> collection) - throws IllegalStateException { + throws IllegalStateException, IllegalArgumentException { boolean modified = false; for (BEANTYPE bean : collection) { // TODO skipping invalid beans - should not allow them in javadoc? |