From e968734dfa87136d7a488156789edf77cac1d7f6 Mon Sep 17 00:00:00 2001 From: Artur Signell Date: Thu, 8 Dec 2011 13:04:33 +0000 Subject: #6603 Added IllegalArgumentException to javadoc and added test svn changeset:22320/svn branch:6.7 --- src/com/vaadin/data/util/AbstractBeanContainer.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'src/com') 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 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 collection) - throws IllegalStateException { + throws IllegalStateException, IllegalArgumentException { boolean modified = false; for (BEANTYPE bean : collection) { // TODO skipping invalid beans - should not allow them in javadoc? -- cgit v1.2.3 From d7061c0cbf63069edd71b9f6502d07b0ea38dcf9 Mon Sep 17 00:00:00 2001 From: Artur Signell Date: Thu, 8 Dec 2011 13:07:34 +0000 Subject: #4925 com.vaadin.tools.ReflectTools javadoc svn changeset:22321/svn branch:6.7 --- src/com/vaadin/tools/ReflectTools.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/com') diff --git a/src/com/vaadin/tools/ReflectTools.java b/src/com/vaadin/tools/ReflectTools.java index 2fc2125842..9f0667f90e 100644 --- a/src/com/vaadin/tools/ReflectTools.java +++ b/src/com/vaadin/tools/ReflectTools.java @@ -15,8 +15,8 @@ import java.lang.reflect.Method; public class ReflectTools { /** * Locates the method in the given class. Returns null if the method is not - * found. This method never throws exceptions. Errors in locating methods - * are considered serious problems and are output to standard error. + * found. Throws an ExceptionInInitializerError if there is a problem + * locating the method as this is mainly called from static blocks. * * @param cls * Class that contains the method -- cgit v1.2.3