diff options
Diffstat (limited to 'src/com/vaadin/data')
| -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? |