From 4c72e0407f28f12ada5ec94ce21603238e1b39c2 Mon Sep 17 00:00:00 2001 From: Michael Minella Date: Fri, 6 Mar 2015 13:01:34 -0600 Subject: [PATCH] Updated afterPropertiesSet checks to verify that fragmentRooteElementNames is not empty BATCH-2352 --- .../springframework/batch/item/xml/StaxEventItemReader.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/StaxEventItemReader.java b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/StaxEventItemReader.java index dd8ea8170..76fab9317 100644 --- a/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/StaxEventItemReader.java +++ b/spring-batch-infrastructure/src/main/java/org/springframework/batch/item/xml/StaxEventItemReader.java @@ -128,10 +128,9 @@ ResourceAwareItemReaderItemStream, InitializingBean { @Override public void afterPropertiesSet() throws Exception { Assert.notNull(unmarshaller, "The Unmarshaller must not be null."); - Assert.notNull(fragmentRootElementNames, "The FragmentRootElementNames must not be null"); - Assert.notNull(fragmentRootElementNames, "The FragmentRootElementNames must not be empty"); + Assert.notEmpty(fragmentRootElementNames, "The FragmentRootElementNames must not be empty"); for (QName fragmentRootElementName : fragmentRootElementNames) { - Assert.hasText(fragmentRootElementName.getLocalPart(), "The FragmentRootElementNames must contain empty elements"); + Assert.hasText(fragmentRootElementName.getLocalPart(), "The FragmentRootElementNames must not contain empty elements"); } }