Warning instead of error for non-present type filter class

Issue: SPR-16356

spring-beans/src/test/java/org/springframework/beans/factory/parsing/CustomProblemReporterTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2016 the original author or authors.
+ * Copyright 2002-2018 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -66,10 +66,9 @@ public class CustomProblemReporterTests {
 
 	private static class CollatingProblemReporter implements ProblemReporter {
 
-		private List<Problem> errors = new ArrayList<>();
+		private final List<Problem> errors = new ArrayList<>();
-
-		private List<Problem> warnings = new ArrayList<>();
 
+		private final List<Problem> warnings = new ArrayList<>();
 
 		@Override
 		public void fatal(Problem problem) {

spring-context/src/main/java/org/springframework/context/annotation/ComponentScanBeanDefinitionParser.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2017 the original author or authors.
+ * Copyright 2002-2018 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -25,7 +25,6 @@ import org.w3c.dom.Node;
 import org.w3c.dom.NodeList;
 
 import org.springframework.beans.BeanUtils;
-import org.springframework.beans.FatalBeanException;
 import org.springframework.beans.factory.config.BeanDefinition;
 import org.springframework.beans.factory.config.BeanDefinitionHolder;
 import org.springframework.beans.factory.parsing.BeanComponentDefinition;
@@ -216,6 +215,10 @@ public class ComponentScanBeanDefinitionParser implements BeanDefinitionParser {
 						scanner.addExcludeFilter(typeFilter);
 					}
 				}
+				catch (ClassNotFoundException ex) {
+					parserContext.getReaderContext().warning(
+							"Ignoring non-present type filter class: " + ex, parserContext.extractSource(element));
+				}
 				catch (Exception ex) {
 					parserContext.getReaderContext().error(
 							ex.getMessage(), parserContext.extractSource(element), ex.getCause());
@@ -225,39 +228,34 @@ public class ComponentScanBeanDefinitionParser implements BeanDefinitionParser {
 	}
 
 	@SuppressWarnings("unchecked")
-	protected TypeFilter createTypeFilter(
+	protected TypeFilter createTypeFilter(Element element, @Nullable ClassLoader classLoader,
-			Element element, @Nullable ClassLoader classLoader, ParserContext parserContext) {
+			ParserContext parserContext) throws ClassNotFoundException {
 
 		String filterType = element.getAttribute(FILTER_TYPE_ATTRIBUTE);
 		String expression = element.getAttribute(FILTER_EXPRESSION_ATTRIBUTE);
 		expression = parserContext.getReaderContext().getEnvironment().resolvePlaceholders(expression);
-		try {
+		if ("annotation".equals(filterType)) {
-			if ("annotation".equals(filterType)) {
+			return new AnnotationTypeFilter((Class<Annotation>) ClassUtils.forName(expression, classLoader));
-				return new AnnotationTypeFilter((Class<Annotation>) ClassUtils.forName(expression, classLoader));
+		}
-			}
+		else if ("assignable".equals(filterType)) {
-			else if ("assignable".equals(filterType)) {
+			return new AssignableTypeFilter(ClassUtils.forName(expression, classLoader));
-				return new AssignableTypeFilter(ClassUtils.forName(expression, classLoader));
+		}
-			}
+		else if ("aspectj".equals(filterType)) {
-			else if ("aspectj".equals(filterType)) {
+			return new AspectJTypeFilter(expression, classLoader);
-				return new AspectJTypeFilter(expression, classLoader);
+		}
-			}
+		else if ("regex".equals(filterType)) {
-			else if ("regex".equals(filterType)) {
+			return new RegexPatternTypeFilter(Pattern.compile(expression));
-				return new RegexPatternTypeFilter(Pattern.compile(expression));
+		}
-			}
+		else if ("custom".equals(filterType)) {
-			else if ("custom".equals(filterType)) {
+			Class<?> filterClass = ClassUtils.forName(expression, classLoader);
-				Class<?> filterClass = ClassUtils.forName(expression, classLoader);
+			if (!TypeFilter.class.isAssignableFrom(filterClass)) {
-				if (!TypeFilter.class.isAssignableFrom(filterClass)) {
+				throw new IllegalArgumentException(
-					throw new IllegalArgumentException(
+						"Class is not assignable to [" + TypeFilter.class.getName() + "]: " + expression);
-							"Class is not assignable to [" + TypeFilter.class.getName() + "]: " + expression);
-				}
-				return (TypeFilter) BeanUtils.instantiateClass(filterClass);
-			}
-			else {
-				throw new IllegalArgumentException("Unsupported filter type: " + filterType);
 			}
+			return (TypeFilter) BeanUtils.instantiateClass(filterClass);
 		}
-		catch (ClassNotFoundException ex) {
+		else {
-			throw new FatalBeanException("Type filter class not found: " + expression, ex);
+			throw new IllegalArgumentException("Unsupported filter type: " + filterType);
 		}
 	}
 

spring-jms/src/main/java/org/springframework/jms/config/JmsListenerContainerParser.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2014 the original author or authors.
+ * Copyright 2002-2018 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -150,7 +150,7 @@ class JmsListenerContainerParser extends AbstractListenerContainerParser {
 				if (!("auto".equals(cache) || "consumer".equals(cache))) {
 					parserContext.getReaderContext().warning(
 							"'cache' attribute not actively supported for listener container of type \"simple\". " +
-									"Effective runtime behavior will be equivalent to \"consumer\" / \"auto\".", containerEle);
+							"Effective runtime behavior will be equivalent to \"consumer\" / \"auto\".", containerEle);
 				}
 			}
 			else {