diff --git a/spring-integration-file/src/main/java/org/springframework/integration/file/config/FileListFilterFactoryBean.java b/spring-integration-file/src/main/java/org/springframework/integration/file/config/FileListFilterFactoryBean.java index 753f6ad854..2a58a8324f 100644 --- a/spring-integration-file/src/main/java/org/springframework/integration/file/config/FileListFilterFactoryBean.java +++ b/spring-integration-file/src/main/java/org/springframework/integration/file/config/FileListFilterFactoryBean.java @@ -79,7 +79,7 @@ public class FileListFilterFactoryBean implements FactoryBean filter; + FileListFilter createdFilter = null; if ((this.filter != null) && (this.filenamePattern != null || this.filenameRegex!=null)) { throw new IllegalArgumentException("The 'filter' reference is mutually exclusive with " + "'filename-pattern' and 'filename-regex' attributes."); @@ -88,46 +88,43 @@ public class FileListFilterFactoryBean implements FactoryBean> filtersNeeded = new ArrayList>(); if (!Boolean.FALSE.equals(this.preventDuplicates)) { //preventDuplicates is either null or true filtersNeeded.add(new AcceptOnceFileListFilter()); } - if (this.filenamePattern!=null){ + if (this.filenamePattern != null) { filtersNeeded.add(new SimplePatternFileListFilter(this.filenamePattern)); } - if (this.filenameRegex!=null){ + if (this.filenameRegex != null) { filtersNeeded.add(new PatternMatchingFileListFilter(this.filenameRegex)); } - if (filtersNeeded.size()==1){ - filter = filtersNeeded.get(0); + if (filtersNeeded.size() == 1) { + createdFilter = filtersNeeded.get(0); } else { - filter = new CompositeFileListFilter(filtersNeeded); + createdFilter = new CompositeFileListFilter(filtersNeeded); } } // no filters are provided else if (Boolean.FALSE.equals(this.preventDuplicates)) { - filter = new AcceptAllFileListFilter(); + createdFilter = new AcceptAllFileListFilter(); } else { // preventDuplicates is either TRUE or NULL - filter = new AcceptOnceFileListFilter(); + createdFilter = new AcceptOnceFileListFilter(); } - if (filter == null) { - filter = new CompositeFileListFilter(); - } - this.result = filter; + this.result = createdFilter; } private CompositeFileListFilter createCompositeWithAcceptOnceFilter(FileListFilter otherFilter) {